Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT
|
|
|
|
// file at the top-level directory of this distribution and at
|
|
|
|
// http://rust-lang.org/COPYRIGHT.
|
|
|
|
//
|
|
|
|
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
|
|
|
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
|
|
|
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
|
|
|
// option. This file may not be copied, modified, or distributed
|
|
|
|
// except according to those terms.
|
|
|
|
|
|
|
|
//! A pass that annotates every item and method with its stability level,
|
|
|
|
//! propagating default levels lexically from parent to children ast nodes.
|
|
|
|
|
2015-10-12 22:01:31 -05:00
|
|
|
pub use self::StabilityLevel::*;
|
|
|
|
|
2015-01-16 12:25:16 -06:00
|
|
|
use lint;
|
2016-03-29 04:54:26 -05:00
|
|
|
use hir::def::Def;
|
2017-08-31 17:08:34 -05:00
|
|
|
use hir::def_id::{CrateNum, CRATE_DEF_INDEX, DefId, LOCAL_CRATE};
|
2016-12-29 15:00:53 -06:00
|
|
|
use ty::{self, TyCtxt};
|
2015-11-19 05:16:35 -06:00
|
|
|
use middle::privacy::AccessLevels;
|
2017-10-28 12:39:00 -05:00
|
|
|
use session::DiagnosticMessageId;
|
2016-11-16 04:52:37 -06:00
|
|
|
use syntax::symbol::Symbol;
|
2017-10-25 18:56:13 -05:00
|
|
|
use syntax_pos::{Span, MultiSpan, DUMMY_SP};
|
2014-07-10 13:17:40 -05:00
|
|
|
use syntax::ast;
|
2015-09-14 04:58:20 -05:00
|
|
|
use syntax::ast::{NodeId, Attribute};
|
2016-05-30 15:55:12 -05:00
|
|
|
use syntax::feature_gate::{GateIssue, emit_feature_err, find_lang_feature_accepted_version};
|
2016-08-22 22:54:53 -05:00
|
|
|
use syntax::attr::{self, Stability, Deprecation};
|
2017-08-31 17:08:34 -05:00
|
|
|
use util::nodemap::{FxHashSet, FxHashMap};
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
|
2016-03-29 00:50:44 -05:00
|
|
|
use hir;
|
2017-08-31 17:08:34 -05:00
|
|
|
use hir::{Item, Generics, StructField, Variant, HirId};
|
2016-11-28 13:00:26 -06:00
|
|
|
use hir::intravisit::{self, Visitor, NestedVisitorMap};
|
2015-07-31 02:04:06 -05:00
|
|
|
|
2014-09-12 05:10:30 -05:00
|
|
|
use std::mem::replace;
|
2015-06-11 19:18:46 -05:00
|
|
|
use std::cmp::Ordering;
|
2014-09-12 05:10:30 -05:00
|
|
|
|
2015-10-12 22:01:31 -05:00
|
|
|
#[derive(RustcEncodable, RustcDecodable, PartialEq, PartialOrd, Clone, Copy, Debug, Eq, Hash)]
|
|
|
|
pub enum StabilityLevel {
|
|
|
|
Unstable,
|
|
|
|
Stable,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl StabilityLevel {
|
|
|
|
pub fn from_attr_level(level: &attr::StabilityLevel) -> Self {
|
|
|
|
if level.is_stable() { Stable } else { Unstable }
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-11-16 10:53:41 -06:00
|
|
|
#[derive(PartialEq)]
|
|
|
|
enum AnnotationKind {
|
|
|
|
// Annotation is required if not inherited from unstable parents
|
2015-11-16 12:01:06 -06:00
|
|
|
Required,
|
2015-11-16 10:53:41 -06:00
|
|
|
// Annotation is useless, reject it
|
2015-11-16 12:01:06 -06:00
|
|
|
Prohibited,
|
2015-11-16 10:53:41 -06:00
|
|
|
// Annotation itself is useless, but it can be propagated to children
|
2015-11-16 12:01:06 -06:00
|
|
|
Container,
|
2015-11-16 10:53:41 -06:00
|
|
|
}
|
|
|
|
|
2016-08-04 08:18:36 -05:00
|
|
|
/// An entry in the `depr_map`.
|
|
|
|
#[derive(Clone)]
|
|
|
|
pub struct DeprecationEntry {
|
|
|
|
/// The metadata of the attribute associated with this entry.
|
|
|
|
pub attr: Deprecation,
|
|
|
|
/// The def id where the attr was originally attached. `None` for non-local
|
|
|
|
/// `DefId`'s.
|
2017-08-31 17:08:34 -05:00
|
|
|
origin: Option<HirId>,
|
2016-08-04 08:18:36 -05:00
|
|
|
}
|
|
|
|
|
2017-09-12 10:07:09 -05:00
|
|
|
impl_stable_hash_for!(struct self::DeprecationEntry {
|
|
|
|
attr,
|
|
|
|
origin
|
|
|
|
});
|
|
|
|
|
2016-08-04 08:18:36 -05:00
|
|
|
impl DeprecationEntry {
|
2017-08-31 17:08:34 -05:00
|
|
|
fn local(attr: Deprecation, id: HirId) -> DeprecationEntry {
|
2016-08-04 08:18:36 -05:00
|
|
|
DeprecationEntry {
|
2017-07-03 13:19:51 -05:00
|
|
|
attr,
|
2017-08-31 17:08:34 -05:00
|
|
|
origin: Some(id),
|
2016-08-04 08:18:36 -05:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
pub fn external(attr: Deprecation) -> DeprecationEntry {
|
2016-08-04 08:18:36 -05:00
|
|
|
DeprecationEntry {
|
2017-07-03 13:19:51 -05:00
|
|
|
attr,
|
2016-08-04 08:18:36 -05:00
|
|
|
origin: None,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
pub fn same_origin(&self, other: &DeprecationEntry) -> bool {
|
|
|
|
match (self.origin, other.origin) {
|
|
|
|
(Some(o1), Some(o2)) => o1 == o2,
|
|
|
|
_ => false
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
/// A stability index, giving the stability level for items and methods.
|
2015-05-25 16:41:27 -05:00
|
|
|
pub struct Index<'tcx> {
|
|
|
|
/// This is mostly a cache, except the stabilities of local items
|
|
|
|
/// are filled by the annotator.
|
2017-08-31 17:08:34 -05:00
|
|
|
stab_map: FxHashMap<HirId, &'tcx Stability>,
|
|
|
|
depr_map: FxHashMap<HirId, DeprecationEntry>,
|
2015-05-25 16:41:27 -05:00
|
|
|
|
|
|
|
/// Maps for each crate whether it is part of the staged API.
|
2016-11-10 11:08:21 -06:00
|
|
|
staged_api: FxHashMap<CrateNum, bool>,
|
|
|
|
|
|
|
|
/// Features enabled for this crate.
|
|
|
|
active_features: FxHashSet<Symbol>,
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2017-09-12 10:07:09 -05:00
|
|
|
impl_stable_hash_for!(struct self::Index<'tcx> {
|
|
|
|
stab_map,
|
|
|
|
depr_map,
|
|
|
|
staged_api,
|
|
|
|
active_features
|
|
|
|
});
|
|
|
|
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
// A private tree-walker for producing an Index.
|
2015-05-25 16:41:27 -05:00
|
|
|
struct Annotator<'a, 'tcx: 'a> {
|
2016-05-02 21:23:22 -05:00
|
|
|
tcx: TyCtxt<'a, 'tcx, 'tcx>,
|
2015-05-25 16:41:27 -05:00
|
|
|
index: &'a mut Index<'tcx>,
|
2015-12-04 10:34:28 -06:00
|
|
|
parent_stab: Option<&'tcx Stability>,
|
2016-08-04 08:18:36 -05:00
|
|
|
parent_depr: Option<DeprecationEntry>,
|
2015-11-16 10:53:41 -06:00
|
|
|
in_trait_impl: bool,
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2015-05-25 16:41:27 -05:00
|
|
|
impl<'a, 'tcx: 'a> Annotator<'a, 'tcx> {
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
// Determine the stability for a node based on its attributes and inherited
|
2014-09-12 05:10:30 -05:00
|
|
|
// stability. The stability is recorded in the index and used as the parent.
|
2015-12-17 11:41:28 -06:00
|
|
|
fn annotate<F>(&mut self, id: NodeId, attrs: &[Attribute],
|
2015-11-16 10:53:41 -06:00
|
|
|
item_sp: Span, kind: AnnotationKind, visit_children: F)
|
2016-11-09 15:45:26 -06:00
|
|
|
where F: FnOnce(&mut Self)
|
2014-12-08 19:26:43 -06:00
|
|
|
{
|
2018-02-14 09:11:02 -06:00
|
|
|
if self.tcx.features().staged_api {
|
2017-08-20 09:49:03 -05:00
|
|
|
// This crate explicitly wants staged API.
|
2015-04-13 20:42:24 -05:00
|
|
|
debug!("annotate(id = {:?}, attrs = {:?})", id, attrs);
|
2015-12-04 10:34:28 -06:00
|
|
|
if let Some(..) = attr::find_deprecation(self.tcx.sess.diagnostic(), attrs, item_sp) {
|
|
|
|
self.tcx.sess.span_err(item_sp, "`#[deprecated]` cannot be used in staged api, \
|
|
|
|
use `#[rustc_deprecated]` instead");
|
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
if let Some(mut stab) = attr::find_stability(self.tcx.sess.diagnostic(),
|
|
|
|
attrs, item_sp) {
|
|
|
|
// Error if prohibited, or can't inherit anything from a container
|
2015-11-16 12:01:06 -06:00
|
|
|
if kind == AnnotationKind::Prohibited ||
|
|
|
|
(kind == AnnotationKind::Container &&
|
|
|
|
stab.level.is_stable() &&
|
2015-12-04 10:34:28 -06:00
|
|
|
stab.rustc_depr.is_none()) {
|
2015-11-16 10:53:41 -06:00
|
|
|
self.tcx.sess.span_err(item_sp, "This stability annotation is useless");
|
|
|
|
}
|
2015-06-06 17:52:28 -05:00
|
|
|
|
2015-11-16 10:53:41 -06:00
|
|
|
debug!("annotate: found {:?}", stab);
|
|
|
|
// If parent is deprecated and we're not, inherit this by merging
|
|
|
|
// deprecated_since and its reason.
|
2015-12-04 10:34:28 -06:00
|
|
|
if let Some(parent_stab) = self.parent_stab {
|
|
|
|
if parent_stab.rustc_depr.is_some() && stab.rustc_depr.is_none() {
|
|
|
|
stab.rustc_depr = parent_stab.rustc_depr.clone()
|
2015-11-16 10:53:41 -06:00
|
|
|
}
|
|
|
|
}
|
2015-06-11 19:18:46 -05:00
|
|
|
|
2015-11-16 10:53:41 -06:00
|
|
|
let stab = self.tcx.intern_stability(stab);
|
|
|
|
|
|
|
|
// Check if deprecated_since < stable_since. If it is,
|
|
|
|
// this is *almost surely* an accident.
|
2016-11-16 04:52:37 -06:00
|
|
|
if let (&Some(attr::RustcDeprecation {since: dep_since, ..}),
|
|
|
|
&attr::Stable {since: stab_since}) = (&stab.rustc_depr, &stab.level) {
|
2015-11-16 10:53:41 -06:00
|
|
|
// Explicit version of iter::order::lt to handle parse errors properly
|
2016-11-16 04:52:37 -06:00
|
|
|
for (dep_v, stab_v) in
|
|
|
|
dep_since.as_str().split(".").zip(stab_since.as_str().split(".")) {
|
2015-11-16 10:53:41 -06:00
|
|
|
if let (Ok(dep_v), Ok(stab_v)) = (dep_v.parse::<u64>(), stab_v.parse()) {
|
|
|
|
match dep_v.cmp(&stab_v) {
|
|
|
|
Ordering::Less => {
|
|
|
|
self.tcx.sess.span_err(item_sp, "An API can't be stabilized \
|
|
|
|
after it is deprecated");
|
|
|
|
break
|
2015-06-11 19:18:46 -05:00
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
Ordering::Equal => continue,
|
|
|
|
Ordering::Greater => break,
|
2015-06-11 19:18:46 -05:00
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
} else {
|
|
|
|
// Act like it isn't less because the question is now nonsensical,
|
|
|
|
// and this makes us not do anything else interesting.
|
|
|
|
self.tcx.sess.span_err(item_sp, "Invalid stability or deprecation \
|
|
|
|
version found");
|
|
|
|
break
|
|
|
|
}
|
2015-06-11 19:18:46 -05:00
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
}
|
2015-06-11 19:18:46 -05:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
self.index.stab_map.insert(hir_id, stab);
|
2015-11-16 10:53:41 -06:00
|
|
|
|
2015-12-04 10:34:28 -06:00
|
|
|
let orig_parent_stab = replace(&mut self.parent_stab, Some(stab));
|
2015-11-16 10:53:41 -06:00
|
|
|
visit_children(self);
|
2015-12-04 10:34:28 -06:00
|
|
|
self.parent_stab = orig_parent_stab;
|
2015-11-16 10:53:41 -06:00
|
|
|
} else {
|
2015-12-04 10:34:28 -06:00
|
|
|
debug!("annotate: not found, parent = {:?}", self.parent_stab);
|
|
|
|
if let Some(stab) = self.parent_stab {
|
2015-11-16 10:53:41 -06:00
|
|
|
if stab.level.is_unstable() {
|
2017-08-31 17:08:34 -05:00
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
self.index.stab_map.insert(hir_id, stab);
|
2015-04-13 20:42:24 -05:00
|
|
|
}
|
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
visit_children(self);
|
2014-09-12 05:10:30 -05:00
|
|
|
}
|
2015-04-13 20:42:24 -05:00
|
|
|
} else {
|
2015-11-16 10:53:41 -06:00
|
|
|
// Emit errors for non-staged-api crates.
|
2015-04-13 20:42:24 -05:00
|
|
|
for attr in attrs {
|
2017-03-03 03:23:59 -06:00
|
|
|
let tag = unwrap_or!(attr.name(), continue);
|
2015-11-20 07:11:20 -06:00
|
|
|
if tag == "unstable" || tag == "stable" || tag == "rustc_deprecated" {
|
2015-04-13 20:42:24 -05:00
|
|
|
attr::mark_used(attr);
|
2015-11-16 10:53:41 -06:00
|
|
|
self.tcx.sess.span_err(attr.span(), "stability attributes may not be used \
|
|
|
|
outside of the standard library");
|
2014-12-17 22:12:41 -06:00
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
2015-12-04 10:34:28 -06:00
|
|
|
|
2017-08-20 09:49:03 -05:00
|
|
|
// Propagate unstability. This can happen even for non-staged-api crates in case
|
|
|
|
// -Zforce-unstable-if-unmarked is set.
|
|
|
|
if let Some(stab) = self.parent_stab {
|
|
|
|
if stab.level.is_unstable() {
|
2017-08-31 17:08:34 -05:00
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
self.index.stab_map.insert(hir_id, stab);
|
2017-08-20 09:49:03 -05:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-12-04 10:34:28 -06:00
|
|
|
if let Some(depr) = attr::find_deprecation(self.tcx.sess.diagnostic(), attrs, item_sp) {
|
|
|
|
if kind == AnnotationKind::Prohibited {
|
|
|
|
self.tcx.sess.span_err(item_sp, "This deprecation annotation is useless");
|
|
|
|
}
|
|
|
|
|
|
|
|
// `Deprecation` is just two pointers, no need to intern it
|
2017-08-31 17:08:34 -05:00
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
let depr_entry = DeprecationEntry::local(depr, hir_id);
|
|
|
|
self.index.depr_map.insert(hir_id, depr_entry.clone());
|
2015-12-04 10:34:28 -06:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
let orig_parent_depr = replace(&mut self.parent_depr,
|
|
|
|
Some(depr_entry));
|
2015-12-04 10:34:28 -06:00
|
|
|
visit_children(self);
|
|
|
|
self.parent_depr = orig_parent_depr;
|
2017-08-31 17:08:34 -05:00
|
|
|
} else if let Some(parent_depr) = self.parent_depr.clone() {
|
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
self.index.depr_map.insert(hir_id, parent_depr);
|
2015-12-04 10:34:28 -06:00
|
|
|
visit_children(self);
|
|
|
|
} else {
|
|
|
|
visit_children(self);
|
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
impl<'a, 'tcx> Visitor<'tcx> for Annotator<'a, 'tcx> {
|
2015-11-17 17:56:13 -06:00
|
|
|
/// Because stability levels are scoped lexically, we want to walk
|
|
|
|
/// nested items in the context of the outer item, so enable
|
|
|
|
/// deep-walking.
|
2016-11-28 13:00:26 -06:00
|
|
|
fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
|
2017-01-25 18:41:06 -06:00
|
|
|
NestedVisitorMap::All(&self.tcx.hir)
|
2016-11-04 17:20:15 -05:00
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_item(&mut self, i: &'tcx Item) {
|
2015-11-16 10:53:41 -06:00
|
|
|
let orig_in_trait_impl = self.in_trait_impl;
|
2015-11-16 12:01:06 -06:00
|
|
|
let mut kind = AnnotationKind::Required;
|
2015-11-16 10:53:41 -06:00
|
|
|
match i.node {
|
|
|
|
// Inherent impls and foreign modules serve only as containers for other items,
|
|
|
|
// they don't have their own stability. They still can be annotated as unstable
|
|
|
|
// and propagate this unstability to children, but this annotation is completely
|
|
|
|
// optional. They inherit stability from their parents when unannotated.
|
2016-08-26 11:23:42 -05:00
|
|
|
hir::ItemImpl(.., None, _, _) | hir::ItemForeignMod(..) => {
|
2015-11-16 10:53:41 -06:00
|
|
|
self.in_trait_impl = false;
|
2015-11-16 12:01:06 -06:00
|
|
|
kind = AnnotationKind::Container;
|
2015-11-16 10:53:41 -06:00
|
|
|
}
|
2016-08-26 11:23:42 -05:00
|
|
|
hir::ItemImpl(.., Some(_), _, _) => {
|
2015-11-16 10:53:41 -06:00
|
|
|
self.in_trait_impl = true;
|
2015-10-01 19:53:28 -05:00
|
|
|
}
|
2015-11-16 10:53:41 -06:00
|
|
|
hir::ItemStruct(ref sd, _) => {
|
|
|
|
if !sd.is_struct() {
|
2015-11-16 12:01:06 -06:00
|
|
|
self.annotate(sd.id(), &i.attrs, i.span, AnnotationKind::Required, |_| {})
|
2015-11-16 10:53:41 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
_ => {}
|
2014-11-11 14:46:47 -06:00
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
|
2015-11-16 10:53:41 -06:00
|
|
|
self.annotate(i.id, &i.attrs, i.span, kind, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_item(v, i)
|
2015-11-16 10:53:41 -06:00
|
|
|
});
|
|
|
|
self.in_trait_impl = orig_in_trait_impl;
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_trait_item(&mut self, ti: &'tcx hir::TraitItem) {
|
2015-11-16 12:01:06 -06:00
|
|
|
self.annotate(ti.id, &ti.attrs, ti.span, AnnotationKind::Required, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_trait_item(v, ti);
|
2015-11-16 10:53:41 -06:00
|
|
|
});
|
2015-03-10 05:28:44 -05:00
|
|
|
}
|
2014-08-05 21:44:21 -05:00
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_impl_item(&mut self, ii: &'tcx hir::ImplItem) {
|
2015-11-16 12:01:06 -06:00
|
|
|
let kind = if self.in_trait_impl {
|
|
|
|
AnnotationKind::Prohibited
|
|
|
|
} else {
|
|
|
|
AnnotationKind::Required
|
|
|
|
};
|
2015-11-16 10:53:41 -06:00
|
|
|
self.annotate(ii.id, &ii.attrs, ii.span, kind, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_impl_item(v, ii);
|
2015-11-16 10:53:41 -06:00
|
|
|
});
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_variant(&mut self, var: &'tcx Variant, g: &'tcx Generics, item_id: NodeId) {
|
2015-11-16 12:01:06 -06:00
|
|
|
self.annotate(var.node.data.id(), &var.node.attrs, var.span, AnnotationKind::Required, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_variant(v, var, g, item_id);
|
2015-11-16 10:53:41 -06:00
|
|
|
})
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_struct_field(&mut self, s: &'tcx StructField) {
|
2016-02-27 02:34:29 -06:00
|
|
|
self.annotate(s.id, &s.attrs, s.span, AnnotationKind::Required, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_struct_field(v, s);
|
2015-11-16 10:53:41 -06:00
|
|
|
});
|
2014-07-10 13:17:40 -05:00
|
|
|
}
|
2014-12-20 12:08:16 -06:00
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_foreign_item(&mut self, i: &'tcx hir::ForeignItem) {
|
2015-11-16 12:01:06 -06:00
|
|
|
self.annotate(i.id, &i.attrs, i.span, AnnotationKind::Required, |v| {
|
2015-11-17 17:56:13 -06:00
|
|
|
intravisit::walk_foreign_item(v, i);
|
2015-11-16 10:53:41 -06:00
|
|
|
});
|
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_macro_def(&mut self, md: &'tcx hir::MacroDef) {
|
2016-11-29 17:36:27 -06:00
|
|
|
self.annotate(md.id, &md.attrs, md.span, AnnotationKind::Required, |_| {});
|
2014-12-20 12:08:16 -06:00
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
struct MissingStabilityAnnotations<'a, 'tcx: 'a> {
|
|
|
|
tcx: TyCtxt<'a, 'tcx, 'tcx>,
|
|
|
|
access_levels: &'a AccessLevels,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl<'a, 'tcx: 'a> MissingStabilityAnnotations<'a, 'tcx> {
|
|
|
|
fn check_missing_stability(&self, id: NodeId, span: Span) {
|
2017-08-31 17:08:34 -05:00
|
|
|
let hir_id = self.tcx.hir.node_to_hir_id(id);
|
|
|
|
let stab = self.tcx.stability().local_stability(hir_id);
|
2016-11-10 11:08:21 -06:00
|
|
|
let is_error = !self.tcx.sess.opts.test &&
|
2017-08-31 17:08:34 -05:00
|
|
|
stab.is_none() &&
|
2016-11-10 11:08:21 -06:00
|
|
|
self.access_levels.is_reachable(id);
|
|
|
|
if is_error {
|
|
|
|
self.tcx.sess.span_err(span, "This node does not have a stability attribute");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
impl<'a, 'tcx> Visitor<'tcx> for MissingStabilityAnnotations<'a, 'tcx> {
|
2016-11-28 13:00:26 -06:00
|
|
|
fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
|
2017-01-25 18:41:06 -06:00
|
|
|
NestedVisitorMap::OnlyBodies(&self.tcx.hir)
|
2016-11-28 11:10:37 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
fn visit_item(&mut self, i: &'tcx Item) {
|
2016-11-10 11:08:21 -06:00
|
|
|
match i.node {
|
|
|
|
// Inherent impls and foreign modules serve only as containers for other items,
|
|
|
|
// they don't have their own stability. They still can be annotated as unstable
|
|
|
|
// and propagate this unstability to children, but this annotation is completely
|
|
|
|
// optional. They inherit stability from their parents when unannotated.
|
|
|
|
hir::ItemImpl(.., None, _, _) | hir::ItemForeignMod(..) => {}
|
|
|
|
|
|
|
|
_ => self.check_missing_stability(i.id, i.span)
|
|
|
|
}
|
|
|
|
|
|
|
|
intravisit::walk_item(self, i)
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_trait_item(&mut self, ti: &'tcx hir::TraitItem) {
|
2016-11-10 11:08:21 -06:00
|
|
|
self.check_missing_stability(ti.id, ti.span);
|
|
|
|
intravisit::walk_trait_item(self, ti);
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_impl_item(&mut self, ii: &'tcx hir::ImplItem) {
|
2017-01-25 18:41:06 -06:00
|
|
|
let impl_def_id = self.tcx.hir.local_def_id(self.tcx.hir.get_parent(ii.id));
|
2016-11-10 11:08:21 -06:00
|
|
|
if self.tcx.impl_trait_ref(impl_def_id).is_none() {
|
|
|
|
self.check_missing_stability(ii.id, ii.span);
|
|
|
|
}
|
|
|
|
intravisit::walk_impl_item(self, ii);
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_variant(&mut self, var: &'tcx Variant, g: &'tcx Generics, item_id: NodeId) {
|
2016-11-10 11:08:21 -06:00
|
|
|
self.check_missing_stability(var.node.data.id(), var.span);
|
|
|
|
intravisit::walk_variant(self, var, g, item_id);
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_struct_field(&mut self, s: &'tcx StructField) {
|
2016-11-10 11:08:21 -06:00
|
|
|
self.check_missing_stability(s.id, s.span);
|
|
|
|
intravisit::walk_struct_field(self, s);
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_foreign_item(&mut self, i: &'tcx hir::ForeignItem) {
|
2016-11-10 11:08:21 -06:00
|
|
|
self.check_missing_stability(i.id, i.span);
|
|
|
|
intravisit::walk_foreign_item(self, i);
|
|
|
|
}
|
|
|
|
|
2016-11-28 11:10:37 -06:00
|
|
|
fn visit_macro_def(&mut self, md: &'tcx hir::MacroDef) {
|
2016-11-29 17:36:27 -06:00
|
|
|
self.check_missing_stability(md.id, md.span);
|
2016-11-10 11:08:21 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-05-02 20:56:42 -05:00
|
|
|
impl<'a, 'tcx> Index<'tcx> {
|
2017-08-31 17:08:34 -05:00
|
|
|
pub fn new(tcx: TyCtxt<'a, 'tcx, 'tcx>) -> Index<'tcx> {
|
|
|
|
let is_staged_api =
|
|
|
|
tcx.sess.opts.debugging_opts.force_unstable_if_unmarked ||
|
2018-02-14 09:11:02 -06:00
|
|
|
tcx.features().staged_api;
|
2017-08-31 17:08:34 -05:00
|
|
|
let mut staged_api = FxHashMap();
|
|
|
|
staged_api.insert(LOCAL_CRATE, is_staged_api);
|
|
|
|
let mut index = Index {
|
|
|
|
staged_api,
|
|
|
|
stab_map: FxHashMap(),
|
|
|
|
depr_map: FxHashMap(),
|
|
|
|
active_features: FxHashSet(),
|
|
|
|
};
|
|
|
|
|
2018-02-14 09:11:02 -06:00
|
|
|
let ref active_lib_features = tcx.features().declared_lib_features;
|
2016-11-10 11:08:21 -06:00
|
|
|
|
|
|
|
// Put the active features into a map for quick lookup
|
2017-08-31 17:08:34 -05:00
|
|
|
index.active_features = active_lib_features.iter().map(|&(ref s, _)| s.clone()).collect();
|
|
|
|
|
|
|
|
{
|
|
|
|
let krate = tcx.hir.krate();
|
|
|
|
let mut annotator = Annotator {
|
|
|
|
tcx,
|
|
|
|
index: &mut index,
|
|
|
|
parent_stab: None,
|
|
|
|
parent_depr: None,
|
|
|
|
in_trait_impl: false,
|
|
|
|
};
|
2016-11-10 11:08:21 -06:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
// If the `-Z force-unstable-if-unmarked` flag is passed then we provide
|
|
|
|
// a parent stability annotation which indicates that this is private
|
|
|
|
// with the `rustc_private` feature. This is intended for use when
|
|
|
|
// compiling librustc crates themselves so we can leverage crates.io
|
|
|
|
// while maintaining the invariant that all sysroot crates are unstable
|
|
|
|
// by default and are unable to be used.
|
|
|
|
if tcx.sess.opts.debugging_opts.force_unstable_if_unmarked {
|
2017-10-05 20:31:59 -05:00
|
|
|
let reason = "this crate is being loaded from the sysroot, an \
|
2017-08-31 17:08:34 -05:00
|
|
|
unstable location; did you mean to load this crate \
|
|
|
|
from crates.io via `Cargo.toml` instead?";
|
|
|
|
let stability = tcx.intern_stability(Stability {
|
|
|
|
level: attr::StabilityLevel::Unstable {
|
|
|
|
reason: Some(Symbol::intern(reason)),
|
|
|
|
issue: 27812,
|
|
|
|
},
|
|
|
|
feature: Symbol::intern("rustc_private"),
|
|
|
|
rustc_depr: None,
|
2017-09-08 13:11:30 -05:00
|
|
|
rustc_const_unstable: None,
|
2017-08-31 17:08:34 -05:00
|
|
|
});
|
|
|
|
annotator.parent_stab = Some(stability);
|
|
|
|
}
|
2017-05-08 15:36:26 -05:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
annotator.annotate(ast::CRATE_NODE_ID,
|
|
|
|
&krate.attrs,
|
|
|
|
krate.span,
|
|
|
|
AnnotationKind::Required,
|
|
|
|
|v| intravisit::walk_crate(v, krate));
|
2017-05-08 15:36:26 -05:00
|
|
|
}
|
2017-08-31 17:08:34 -05:00
|
|
|
return index
|
|
|
|
}
|
2017-05-08 15:36:26 -05:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
pub fn local_stability(&self, id: HirId) -> Option<&'tcx Stability> {
|
|
|
|
self.stab_map.get(&id).cloned()
|
2015-02-03 09:46:08 -06:00
|
|
|
}
|
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
pub fn local_deprecation_entry(&self, id: HirId) -> Option<DeprecationEntry> {
|
|
|
|
self.depr_map.get(&id).cloned()
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
2014-06-26 13:37:39 -05:00
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
|
2015-01-14 17:20:14 -06:00
|
|
|
/// Cross-references the feature names of unstable APIs with enabled
|
2016-11-10 11:08:21 -06:00
|
|
|
/// features and possibly prints errors.
|
|
|
|
pub fn check_unstable_api_usage<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>) {
|
|
|
|
let mut checker = Checker { tcx: tcx };
|
2017-04-18 04:57:39 -05:00
|
|
|
tcx.hir.krate().visit_all_item_likes(&mut checker.as_deep_visitor());
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
struct Checker<'a, 'tcx: 'a> {
|
2016-05-02 21:23:22 -05:00
|
|
|
tcx: TyCtxt<'a, 'tcx, 'tcx>,
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
impl<'a, 'gcx, 'tcx> TyCtxt<'a, 'gcx, 'tcx> {
|
2016-12-29 15:00:53 -06:00
|
|
|
// (See issue #38412)
|
2016-09-28 18:30:53 -05:00
|
|
|
fn skip_stability_check_due_to_privacy(self, mut def_id: DefId) -> bool {
|
|
|
|
// Check if `def_id` is a trait method.
|
2017-04-27 13:27:16 -05:00
|
|
|
match self.describe_def(def_id) {
|
2016-09-28 18:30:53 -05:00
|
|
|
Some(Def::Method(_)) |
|
|
|
|
Some(Def::AssociatedTy(_)) |
|
|
|
|
Some(Def::AssociatedConst(_)) => {
|
|
|
|
match self.associated_item(def_id).container {
|
|
|
|
ty::TraitContainer(trait_def_id) => {
|
|
|
|
// Trait methods do not declare visibility (even
|
|
|
|
// for visibility info in cstore). Use containing
|
|
|
|
// trait instead, so methods of pub traits are
|
|
|
|
// themselves considered pub.
|
|
|
|
def_id = trait_def_id;
|
|
|
|
}
|
|
|
|
_ => {}
|
2016-12-29 15:00:53 -06:00
|
|
|
}
|
|
|
|
}
|
2016-09-28 18:30:53 -05:00
|
|
|
_ => {}
|
|
|
|
}
|
|
|
|
|
2017-08-31 10:07:39 -05:00
|
|
|
let visibility = self.visibility(def_id);
|
2016-12-29 15:00:53 -06:00
|
|
|
|
|
|
|
match visibility {
|
|
|
|
// must check stability for pub items.
|
|
|
|
ty::Visibility::Public => false,
|
|
|
|
|
|
|
|
// these are not visible outside crate; therefore
|
|
|
|
// stability markers are irrelevant, if even present.
|
|
|
|
ty::Visibility::Restricted(..) |
|
|
|
|
ty::Visibility::Invisible => true,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
pub fn check_stability(self, def_id: DefId, id: NodeId, span: Span) {
|
2017-03-16 23:04:41 -05:00
|
|
|
if span.allows_unstable() {
|
2016-11-10 11:08:21 -06:00
|
|
|
debug!("stability: \
|
|
|
|
skipping span={:?} since it is internal", span);
|
2015-12-04 10:34:28 -06:00
|
|
|
return;
|
|
|
|
}
|
2016-11-10 11:08:21 -06:00
|
|
|
|
2017-11-01 18:34:44 -05:00
|
|
|
let lint_deprecated = |def_id: DefId, note: Option<Symbol>| {
|
|
|
|
let path = self.item_path_str(def_id);
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
let msg = if let Some(note) = note {
|
2017-11-01 18:34:44 -05:00
|
|
|
format!("use of deprecated item '{}': {}", path, note)
|
2016-11-10 11:08:21 -06:00
|
|
|
} else {
|
2017-11-01 18:34:44 -05:00
|
|
|
format!("use of deprecated item '{}'", path)
|
2016-11-10 11:08:21 -06:00
|
|
|
};
|
|
|
|
|
2017-07-26 23:51:09 -05:00
|
|
|
self.lint_node(lint::builtin::DEPRECATED, id, span, &msg);
|
2016-11-10 11:08:21 -06:00
|
|
|
};
|
|
|
|
|
|
|
|
// Deprecated attributes apply in-crate and cross-crate.
|
|
|
|
if let Some(depr_entry) = self.lookup_deprecation_entry(def_id) {
|
|
|
|
let skip = if id == ast::DUMMY_NODE_ID {
|
|
|
|
true
|
|
|
|
} else {
|
2017-01-25 18:41:06 -06:00
|
|
|
let parent_def_id = self.hir.local_def_id(self.hir.get_parent(id));
|
2016-11-10 11:08:21 -06:00
|
|
|
self.lookup_deprecation_entry(parent_def_id).map_or(false, |parent_depr| {
|
|
|
|
parent_depr.same_origin(&depr_entry)
|
|
|
|
})
|
|
|
|
};
|
|
|
|
|
|
|
|
if !skip {
|
2017-11-01 18:34:44 -05:00
|
|
|
lint_deprecated(def_id, depr_entry.attr.note);
|
2016-11-10 11:08:21 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-05-08 15:36:26 -05:00
|
|
|
let is_staged_api = self.lookup_stability(DefId {
|
|
|
|
index: CRATE_DEF_INDEX,
|
|
|
|
..def_id
|
|
|
|
}).is_some();
|
2016-11-10 11:08:21 -06:00
|
|
|
if !is_staged_api {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
let stability = self.lookup_stability(def_id);
|
|
|
|
debug!("stability: \
|
|
|
|
inspecting def_id={:?} span={:?} of stability={:?}", def_id, span, stability);
|
|
|
|
|
|
|
|
if let Some(&Stability{rustc_depr: Some(attr::RustcDeprecation { reason, .. }), ..})
|
|
|
|
= stability {
|
|
|
|
if id != ast::DUMMY_NODE_ID {
|
2017-11-01 18:34:44 -05:00
|
|
|
lint_deprecated(def_id, Some(reason));
|
2016-11-10 11:08:21 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-01-14 17:20:14 -06:00
|
|
|
// Only the cross-crate scenario matters when checking unstable APIs
|
2016-11-10 11:08:21 -06:00
|
|
|
let cross_crate = !def_id.is_local();
|
2015-09-28 19:46:01 -05:00
|
|
|
if !cross_crate {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
2016-12-29 15:00:53 -06:00
|
|
|
// Issue 38412: private items lack stability markers.
|
|
|
|
if self.skip_stability_check_due_to_privacy(def_id) {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
match stability {
|
|
|
|
Some(&Stability { level: attr::Unstable {ref reason, issue}, ref feature, .. }) => {
|
2017-08-31 17:08:34 -05:00
|
|
|
if self.stability().active_features.contains(feature) {
|
2017-05-08 15:36:26 -05:00
|
|
|
return
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
2017-05-08 15:36:26 -05:00
|
|
|
|
|
|
|
// When we're compiling the compiler itself we may pull in
|
|
|
|
// crates from crates.io, but those crates may depend on other
|
|
|
|
// crates also pulled in from crates.io. We want to ideally be
|
|
|
|
// able to compile everything without requiring upstream
|
|
|
|
// modifications, so in the case that this looks like a
|
|
|
|
// rustc_private crate (e.g. a compiler crate) and we also have
|
|
|
|
// the `-Z force-unstable-if-unmarked` flag present (we're
|
|
|
|
// compiling a compiler crate), then let this missing feature
|
|
|
|
// annotation slide.
|
|
|
|
if *feature == "rustc_private" && issue == 27812 {
|
|
|
|
if self.sess.opts.debugging_opts.force_unstable_if_unmarked {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
let msg = match *reason {
|
|
|
|
Some(ref r) => format!("use of unstable library feature '{}': {}",
|
|
|
|
feature.as_str(), &r),
|
|
|
|
None => format!("use of unstable library feature '{}'", &feature)
|
|
|
|
};
|
2017-10-25 18:56:13 -05:00
|
|
|
|
2017-10-26 11:34:39 -05:00
|
|
|
|
2017-10-25 18:56:13 -05:00
|
|
|
let msp: MultiSpan = span.into();
|
|
|
|
let cm = &self.sess.parse_sess.codemap();
|
2017-10-28 12:39:00 -05:00
|
|
|
let span_key = msp.primary_span().and_then(|sp: Span|
|
|
|
|
if sp != DUMMY_SP {
|
|
|
|
let file = cm.lookup_char_pos(sp.lo()).file;
|
2017-12-14 01:09:19 -06:00
|
|
|
if file.name.is_macros() {
|
2017-10-25 18:56:13 -05:00
|
|
|
None
|
2017-10-28 12:39:00 -05:00
|
|
|
} else {
|
|
|
|
Some(span)
|
2017-10-25 18:56:13 -05:00
|
|
|
}
|
2017-10-28 12:39:00 -05:00
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
);
|
2017-10-25 18:56:13 -05:00
|
|
|
|
2017-10-28 12:39:00 -05:00
|
|
|
let error_id = (DiagnosticMessageId::StabilityId(issue), span_key, msg.clone());
|
|
|
|
let fresh = self.sess.one_time_diagnostics.borrow_mut().insert(error_id);
|
2017-10-26 11:34:39 -05:00
|
|
|
if fresh {
|
|
|
|
emit_feature_err(&self.sess.parse_sess, &feature.as_str(), span,
|
|
|
|
GateIssue::Library(Some(issue)), &msg);
|
2017-10-25 18:56:13 -05:00
|
|
|
}
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
2016-11-10 11:08:21 -06:00
|
|
|
Some(_) => {
|
2015-01-14 17:20:14 -06:00
|
|
|
// Stable APIs are always ok to call and deprecated APIs are
|
2016-11-10 11:08:21 -06:00
|
|
|
// handled by the lint emitting logic above.
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
|
|
|
None => {
|
2016-11-30 08:23:11 -06:00
|
|
|
span_bug!(span, "encountered unmarked API");
|
2015-01-14 17:20:14 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
impl<'a, 'tcx> Visitor<'tcx> for Checker<'a, 'tcx> {
|
2016-11-28 13:00:26 -06:00
|
|
|
/// Because stability levels are scoped lexically, we want to walk
|
|
|
|
/// nested items in the context of the outer item, so enable
|
|
|
|
/// deep-walking.
|
|
|
|
fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
|
2017-01-25 18:41:06 -06:00
|
|
|
NestedVisitorMap::OnlyBodies(&self.tcx.hir)
|
2016-10-28 15:58:32 -05:00
|
|
|
}
|
|
|
|
|
2016-11-09 15:45:26 -06:00
|
|
|
fn visit_item(&mut self, item: &'tcx hir::Item) {
|
2016-11-10 11:08:21 -06:00
|
|
|
match item.node {
|
|
|
|
hir::ItemExternCrate(_) => {
|
|
|
|
// compiler-generated `extern crate` items have a dummy span.
|
|
|
|
if item.span == DUMMY_SP { return }
|
|
|
|
|
2017-09-08 15:51:57 -05:00
|
|
|
let def_id = self.tcx.hir.local_def_id(item.id);
|
|
|
|
let cnum = match self.tcx.extern_mod_stmt_cnum(def_id) {
|
2016-11-10 11:08:21 -06:00
|
|
|
Some(cnum) => cnum,
|
|
|
|
None => return,
|
|
|
|
};
|
|
|
|
let def_id = DefId { krate: cnum, index: CRATE_DEF_INDEX };
|
|
|
|
self.tcx.check_stability(def_id, item.id, item.span);
|
2015-02-17 15:56:06 -06:00
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
// For implementations of traits, check the stability of each item
|
|
|
|
// individually as it's possible to have a stable trait with unstable
|
|
|
|
// items.
|
|
|
|
hir::ItemImpl(.., Some(ref t), _, ref impl_item_refs) => {
|
|
|
|
if let Def::Trait(trait_did) = t.path.def {
|
|
|
|
for impl_item_ref in impl_item_refs {
|
2017-01-25 18:41:06 -06:00
|
|
|
let impl_item = self.tcx.hir.impl_item(impl_item_ref.id);
|
2016-11-10 11:08:21 -06:00
|
|
|
let trait_item_def_id = self.tcx.associated_items(trait_did)
|
|
|
|
.find(|item| item.name == impl_item.name).map(|item| item.def_id);
|
|
|
|
if let Some(def_id) = trait_item_def_id {
|
|
|
|
// Pass `DUMMY_NODE_ID` to skip deprecation warnings.
|
|
|
|
self.tcx.check_stability(def_id, ast::DUMMY_NODE_ID, impl_item.span);
|
2016-09-05 17:26:02 -05:00
|
|
|
}
|
|
|
|
}
|
2016-11-10 11:08:21 -06:00
|
|
|
}
|
2015-02-25 05:34:21 -06:00
|
|
|
}
|
2015-01-14 17:20:14 -06:00
|
|
|
|
2017-05-17 16:22:52 -05:00
|
|
|
// There's no good place to insert stability check for non-Copy unions,
|
|
|
|
// so semi-randomly perform it here in stability.rs
|
2018-02-14 09:11:02 -06:00
|
|
|
hir::ItemUnion(..) if !self.tcx.features().untagged_unions => {
|
2017-05-17 16:22:52 -05:00
|
|
|
let def_id = self.tcx.hir.local_def_id(item.id);
|
|
|
|
let adt_def = self.tcx.adt_def(def_id);
|
|
|
|
let ty = self.tcx.type_of(def_id);
|
|
|
|
|
|
|
|
if adt_def.has_dtor(self.tcx) {
|
|
|
|
emit_feature_err(&self.tcx.sess.parse_sess,
|
|
|
|
"untagged_unions", item.span, GateIssue::Language,
|
|
|
|
"unions with `Drop` implementations are unstable");
|
|
|
|
} else {
|
|
|
|
let param_env = self.tcx.param_env(def_id);
|
|
|
|
if !param_env.can_type_implement_copy(self.tcx, ty, item.span).is_ok() {
|
|
|
|
emit_feature_err(&self.tcx.sess.parse_sess,
|
|
|
|
"untagged_unions", item.span, GateIssue::Language,
|
|
|
|
"unions with non-`Copy` fields are unstable");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
_ => (/* pass */)
|
2015-02-25 05:34:21 -06:00
|
|
|
}
|
2016-11-10 11:08:21 -06:00
|
|
|
intravisit::walk_item(self, item);
|
2015-02-25 05:34:21 -06:00
|
|
|
}
|
2016-10-26 21:17:42 -05:00
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
fn visit_path(&mut self, path: &'tcx hir::Path, id: ast::NodeId) {
|
|
|
|
match path.def {
|
2017-04-29 06:39:47 -05:00
|
|
|
Def::Local(..) | Def::Upvar(..) |
|
2016-11-10 11:08:21 -06:00
|
|
|
Def::PrimTy(..) | Def::SelfTy(..) | Def::Err => {}
|
|
|
|
_ => self.tcx.check_stability(path.def.def_id(), id, path.span)
|
|
|
|
}
|
|
|
|
intravisit::walk_path(self, path)
|
2016-10-26 21:17:42 -05:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-11-10 11:08:21 -06:00
|
|
|
impl<'a, 'gcx, 'tcx> TyCtxt<'a, 'gcx, 'tcx> {
|
2016-05-11 00:48:12 -05:00
|
|
|
pub fn lookup_deprecation(self, id: DefId) -> Option<Deprecation> {
|
2016-08-04 08:18:36 -05:00
|
|
|
self.lookup_deprecation_entry(id).map(|depr| depr.attr)
|
|
|
|
}
|
Add stability inheritance
This commit makes several changes to the stability index infrastructure:
* Stability levels are now inherited lexically, i.e., each item's
stability level becomes the default for any nested items.
* The computed stability level for an item is stored as part of the
metadata. When using an item from an external crate, this data is
looked up and cached.
* The stability lint works from the computed stability level, rather
than manual stability attribute annotations. However, the lint still
checks only a limited set of item uses (e.g., it does not check every
component of a path on import). This will be addressed in a later PR,
as part of issue #8962.
* The stability lint only applies to items originating from external
crates, since the stability index is intended as a promise to
downstream crates.
* The "experimental" lint is now _allow_ by default. This is because
almost all existing crates have been marked "experimental", pending
library stabilization. With inheritance in place, this would generate
a massive explosion of warnings for every Rust program.
The lint should be changed back to deny-by-default after library
stabilization is complete.
* The "deprecated" lint still warns by default.
The net result: we can begin tracking stability index for the standard
libraries as we stabilize, without impacting most clients.
Closes #13540.
2014-06-11 19:23:11 -05:00
|
|
|
}
|
Preliminary feature staging
This partially implements the feature staging described in the
[release channel RFC][rc]. It does not yet fully conform to the RFC as
written, but does accomplish its goals sufficiently for the 1.0 alpha
release.
It has three primary user-visible effects:
* On the nightly channel, use of unstable APIs generates a warning.
* On the beta channel, use of unstable APIs generates a warning.
* On the beta channel, use of feature gates generates a warning.
Code that does not trigger these warnings is considered 'stable',
modulo pre-1.0 bugs.
Disabling the warnings for unstable APIs continues to be done in the
existing (i.e. old) style, via `#[allow(...)]`, not that specified in
the RFC. I deem this marginally acceptable since any code that must do
this is not using the stable dialect of Rust.
Use of feature gates is itself gated with the new 'unstable_features'
lint, on nightly set to 'allow', and on beta 'warn'.
The attribute scheme used here corresponds to an older version of the
RFC, with the `#[staged_api]` crate attribute toggling the staging
behavior of the stability attributes, but the user impact is only
in-tree so I'm not concerned about having to make design changes later
(and I may ultimately prefer the scheme here after all, with the
`#[staged_api]` crate attribute).
Since the Rust codebase itself makes use of unstable features the
compiler and build system to a midly elaborate dance to allow it to
bootstrap while disobeying these lints (which would otherwise be
errors because Rust builds with `-D warnings`).
This patch includes one significant hack that causes a
regression. Because the `format_args!` macro emits calls to unstable
APIs it would trigger the lint. I added a hack to the lint to make it
not trigger, but this in turn causes arguments to `println!` not to be
checked for feature gates. I don't presently understand macro
expansion well enough to fix. This is bug #20661.
Closes #16678
[rc]: https://github.com/rust-lang/rfcs/blob/master/text/0507-release-channels.md
2015-01-06 08:26:08 -06:00
|
|
|
|
2015-01-14 17:20:14 -06:00
|
|
|
/// Given the list of enabled features that were not language features (i.e. that
|
|
|
|
/// were expected to be library features), and the list of features used from
|
|
|
|
/// libraries, identify activated features that don't exist and error about them.
|
2017-03-23 14:13:29 -05:00
|
|
|
pub fn check_unused_or_stable_features<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>) {
|
2017-04-24 09:23:36 -05:00
|
|
|
let access_levels = &tcx.privacy_access_levels(LOCAL_CRATE);
|
2017-03-23 14:13:29 -05:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
if tcx.stability().staged_api[&LOCAL_CRATE] {
|
2017-01-25 18:41:06 -06:00
|
|
|
let krate = tcx.hir.krate();
|
2016-11-10 11:08:21 -06:00
|
|
|
let mut missing = MissingStabilityAnnotations {
|
2017-07-03 13:19:51 -05:00
|
|
|
tcx,
|
|
|
|
access_levels,
|
2016-11-10 11:08:21 -06:00
|
|
|
};
|
|
|
|
missing.check_missing_stability(ast::CRATE_NODE_ID, krate.span);
|
|
|
|
intravisit::walk_crate(&mut missing, krate);
|
2016-11-28 13:00:26 -06:00
|
|
|
krate.visit_all_item_likes(&mut missing.as_deep_visitor());
|
2016-11-10 11:08:21 -06:00
|
|
|
}
|
|
|
|
|
2018-02-14 09:11:02 -06:00
|
|
|
let ref declared_lib_features = tcx.features().declared_lib_features;
|
2016-11-16 04:52:37 -06:00
|
|
|
let mut remaining_lib_features: FxHashMap<Symbol, Span>
|
2015-02-02 22:25:42 -06:00
|
|
|
= declared_lib_features.clone().into_iter().collect();
|
2017-03-17 18:41:09 -05:00
|
|
|
remaining_lib_features.remove(&Symbol::intern("proc_macro"));
|
2015-02-02 22:25:42 -06:00
|
|
|
|
2018-02-14 09:11:02 -06:00
|
|
|
for &(ref stable_lang_feature, span) in &tcx.features().declared_stable_lang_features {
|
2016-11-16 04:52:37 -06:00
|
|
|
let version = find_lang_feature_accepted_version(&stable_lang_feature.as_str())
|
2016-05-30 15:55:12 -05:00
|
|
|
.expect("unexpectedly couldn't find version feature was stabilized");
|
2017-07-26 23:51:09 -05:00
|
|
|
tcx.lint_node(lint::builtin::STABLE_FEATURES,
|
2015-02-02 22:25:42 -06:00
|
|
|
ast::CRATE_NODE_ID,
|
|
|
|
span,
|
2017-07-26 23:51:09 -05:00
|
|
|
&format_stable_since_msg(version));
|
2015-02-02 22:25:42 -06:00
|
|
|
}
|
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
// FIXME(#44232) the `used_features` table no longer exists, so we don't
|
|
|
|
// lint about unknown or unused features. We should reenable
|
|
|
|
// this one day!
|
|
|
|
//
|
|
|
|
// let index = tcx.stability();
|
|
|
|
// for (used_lib_feature, level) in &index.used_features {
|
|
|
|
// remaining_lib_features.remove(used_lib_feature);
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// for &span in remaining_lib_features.values() {
|
|
|
|
// tcx.lint_node(lint::builtin::UNUSED_FEATURES,
|
|
|
|
// ast::CRATE_NODE_ID,
|
|
|
|
// span,
|
|
|
|
// "unused or unknown feature");
|
|
|
|
// }
|
|
|
|
}
|
2015-01-16 12:25:16 -06:00
|
|
|
|
2017-08-31 17:08:34 -05:00
|
|
|
fn format_stable_since_msg(version: &str) -> String {
|
|
|
|
format!("this feature has been stable since {}. Attribute no longer needed", version)
|
Preliminary feature staging
This partially implements the feature staging described in the
[release channel RFC][rc]. It does not yet fully conform to the RFC as
written, but does accomplish its goals sufficiently for the 1.0 alpha
release.
It has three primary user-visible effects:
* On the nightly channel, use of unstable APIs generates a warning.
* On the beta channel, use of unstable APIs generates a warning.
* On the beta channel, use of feature gates generates a warning.
Code that does not trigger these warnings is considered 'stable',
modulo pre-1.0 bugs.
Disabling the warnings for unstable APIs continues to be done in the
existing (i.e. old) style, via `#[allow(...)]`, not that specified in
the RFC. I deem this marginally acceptable since any code that must do
this is not using the stable dialect of Rust.
Use of feature gates is itself gated with the new 'unstable_features'
lint, on nightly set to 'allow', and on beta 'warn'.
The attribute scheme used here corresponds to an older version of the
RFC, with the `#[staged_api]` crate attribute toggling the staging
behavior of the stability attributes, but the user impact is only
in-tree so I'm not concerned about having to make design changes later
(and I may ultimately prefer the scheme here after all, with the
`#[staged_api]` crate attribute).
Since the Rust codebase itself makes use of unstable features the
compiler and build system to a midly elaborate dance to allow it to
bootstrap while disobeying these lints (which would otherwise be
errors because Rust builds with `-D warnings`).
This patch includes one significant hack that causes a
regression. Because the `format_args!` macro emits calls to unstable
APIs it would trigger the lint. I added a hack to the lint to make it
not trigger, but this in turn causes arguments to `println!` not to be
checked for feature gates. I don't presently understand macro
expansion well enough to fix. This is bug #20661.
Closes #16678
[rc]: https://github.com/rust-lang/rfcs/blob/master/text/0507-release-channels.md
2015-01-06 08:26:08 -06:00
|
|
|
}
|