use rustc_span::BytePos;
use syntax::ast::{self, AttrKind, AttrStyle, AttrVec, Attribute, Ident, DUMMY_NODE_ID};
use syntax::ast::{AssocItem, AssocItemKind, Item, ItemKind, UseTree, UseTreeKind};
-use syntax::ast::{Async, Const, Defaultness, IsAuto, PathSegment, StrLit, Unsafe};
+use syntax::ast::{Async, Const, Defaultness, IsAuto, PathSegment, Unsafe};
use syntax::ast::{BindingMode, Block, FnDecl, FnSig, Mac, MacArgs, MacDelimiter, Param, SelfKind};
use syntax::ast::{EnumDef, Generics, StructField, TraitRef, Ty, TyKind, Variant, VariantData};
use syntax::ast::{FnHeader, ForeignItem, ForeignItemKind, Mutability, Visibility, VisibilityKind};
use log::debug;
use std::mem;
-pub(super) type ItemInfo = (Ident, ItemKind, Option<Vec<Attribute>>);
+pub(super) type ItemInfo = (Ident, ItemKind);
impl<'a> Parser<'a> {
pub fn parse_item(&mut self) -> PResult<'a, Option<P<Item>>> {
});
let lo = self.token.span;
-
let vis = self.parse_visibility(FollowedByType::No)?;
- if self.eat_keyword(kw::Use) {
- // USE ITEM
- let item_ = ItemKind::Use(P(self.parse_use_tree()?));
- self.expect_semi()?;
+ if let Some((ident, kind)) = self.parse_item_kind(&mut attrs, macros_allowed, lo, &vis)? {
+ return Ok(Some(self.mk_item(lo.to(self.prev_span), ident, kind, vis, attrs)));
+ }
- let span = lo.to(self.prev_span);
- let item = self.mk_item(span, Ident::invalid(), item_, vis, attrs);
- return Ok(Some(item));
+ // FAILURE TO PARSE ITEM
+ match vis.node {
+ VisibilityKind::Inherited => {}
+ _ => {
+ self.struct_span_err(vis.span, "unmatched visibility `pub`")
+ .span_label(vis.span, "the unmatched visibility")
+ .help("you likely meant to define an item, e.g., `pub fn foo() {}`")
+ .emit();
+ }
}
- if self.is_fn_front_matter() {
- // FUNCTION ITEM
- let (ident, sig, generics, body) =
- self.parse_fn(&mut false, &mut attrs, &ParamCfg::FREE)?;
- let kind = ItemKind::Fn(sig, generics, body);
- return self.mk_item_with_info(attrs, lo, vis, (ident, kind, None));
+ if !attributes_allowed && !attrs.is_empty() {
+ self.expected_item_err(&attrs)?;
}
+ Ok(None)
+ }
- if self.eat_keyword(kw::Extern) {
+ /// Parses one of the items allowed by the flags.
+ fn parse_item_kind(
+ &mut self,
+ attrs: &mut Vec<Attribute>,
+ macros_allowed: bool,
+ lo: Span,
+ vis: &Visibility,
+ ) -> PResult<'a, Option<ItemInfo>> {
+ let info = if self.eat_keyword(kw::Use) {
+ // USE ITEM
+ let tree = self.parse_use_tree()?;
+ self.expect_semi()?;
+ (Ident::invalid(), ItemKind::Use(P(tree)))
+ } else if self.check_fn_front_matter() {
+ // FUNCTION ITEM
+ let (ident, sig, generics, body) = self.parse_fn(&mut false, attrs, |_| true)?;
+ (ident, ItemKind::Fn(sig, generics, body))
+ } else if self.eat_keyword(kw::Extern) {
if self.eat_keyword(kw::Crate) {
// EXTERN CRATE
- return Ok(Some(self.parse_item_extern_crate(lo, vis, attrs)?));
+ self.parse_item_extern_crate()?
+ } else {
+ // EXTERN BLOCK
+ self.parse_item_foreign_mod(attrs)?
}
- // EXTERN BLOCK
- let abi = self.parse_abi();
- return Ok(Some(self.parse_item_foreign_mod(lo, abi, vis, attrs)?));
- }
-
- if self.is_static_global() {
+ } else if self.is_static_global() {
// STATIC ITEM
- self.bump();
+ self.bump(); // `static`
let m = self.parse_mutability();
- let info = self.parse_item_const(Some(m))?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if let Const::Yes(const_span) = self.parse_constness() {
+ self.parse_item_const(Some(m))?
+ } else if let Const::Yes(const_span) = self.parse_constness() {
// CONST ITEM
if self.eat_keyword(kw::Mut) {
let prev_span = self.prev_span;
.emit();
}
- let info = self.parse_item_const(None)?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.check_keyword(kw::Unsafe) && self.is_keyword_ahead(1, &[kw::Trait, kw::Auto]) {
+ self.parse_item_const(None)?
+ } else if self.check_keyword(kw::Unsafe) && self.is_keyword_ahead(1, &[kw::Trait, kw::Auto])
+ {
// UNSAFE TRAIT ITEM
let unsafety = self.parse_unsafety();
- let info = self.parse_item_trait(lo, unsafety)?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.check_keyword(kw::Impl)
+ self.parse_item_trait(attrs, lo, unsafety)?
+ } else if self.check_keyword(kw::Impl)
|| self.check_keyword(kw::Unsafe) && self.is_keyword_ahead(1, &[kw::Impl])
|| self.check_keyword(kw::Default) && self.is_keyword_ahead(1, &[kw::Impl, kw::Unsafe])
{
let defaultness = self.parse_defaultness();
let unsafety = self.parse_unsafety();
self.expect_keyword(kw::Impl)?;
- let info = self.parse_item_impl(unsafety, defaultness)?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.eat_keyword(kw::Mod) {
+ self.parse_item_impl(attrs, unsafety, defaultness)?
+ } else if self.eat_keyword(kw::Mod) {
// MODULE ITEM
- let info = self.parse_item_mod(&attrs[..])?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.eat_keyword(kw::Type) {
+ self.parse_item_mod(attrs)?
+ } else if self.eat_keyword(kw::Type) {
// TYPE ITEM
let (ident, ty, generics) = self.parse_type_alias()?;
- let kind = ItemKind::TyAlias(ty, generics);
- return self.mk_item_with_info(attrs, lo, vis, (ident, kind, None));
- }
-
- if self.eat_keyword(kw::Enum) {
+ (ident, ItemKind::TyAlias(ty, generics))
+ } else if self.eat_keyword(kw::Enum) {
// ENUM ITEM
- let info = self.parse_item_enum()?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.check_keyword(kw::Trait)
+ self.parse_item_enum()?
+ } else if self.check_keyword(kw::Trait)
|| (self.check_keyword(kw::Auto) && self.is_keyword_ahead(1, &[kw::Trait]))
{
// TRAIT ITEM
- let info = self.parse_item_trait(lo, Unsafe::No)?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.eat_keyword(kw::Struct) {
+ self.parse_item_trait(attrs, lo, Unsafe::No)?
+ } else if self.eat_keyword(kw::Struct) {
// STRUCT ITEM
- let info = self.parse_item_struct()?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
-
- if self.is_union_item() {
+ self.parse_item_struct()?
+ } else if self.is_union_item() {
// UNION ITEM
- self.bump();
- let info = self.parse_item_union()?;
- return self.mk_item_with_info(attrs, lo, vis, info);
- }
+ self.bump(); // `union`
+ self.parse_item_union()?
+ } else if self.eat_keyword(kw::Macro) {
+ // MACROS 2.0 ITEM
+ self.parse_item_decl_macro(lo)?
+ } else if self.is_macro_rules_item() {
+ // MACRO_RULES ITEM
+ self.parse_item_macro_rules(vis)?
+ } else if vis.node.is_pub()
+ && self.check_ident()
+ && self.look_ahead(1, |t| *t != token::Not)
+ {
+ self.recover_missing_kw_before_item()?;
+ return Ok(None);
+ } else if macros_allowed && self.token.is_path_start() {
+ // MACRO INVOCATION ITEM
+ self.parse_item_macro(vis)?
+ } else {
+ return Ok(None);
+ };
+ Ok(Some(info))
+ }
- if let Some(macro_def) = self.eat_macro_def(&attrs, &vis, lo)? {
- return Ok(Some(macro_def));
- }
+ /// Recover on encountering a struct or method definition where the user
+ /// forgot to add the `struct` or `fn` keyword after writing `pub`: `pub S {}`.
+ fn recover_missing_kw_before_item(&mut self) -> PResult<'a, ()> {
+ // Space between `pub` keyword and the identifier
+ //
+ // pub S {}
+ // ^^^ `sp` points here
+ let sp = self.prev_span.between(self.token.span);
+ let full_sp = self.prev_span.to(self.token.span);
+ let ident_sp = self.token.span;
+ if self.look_ahead(1, |t| *t == token::OpenDelim(token::Brace)) {
+ // possible public struct definition where `struct` was forgotten
+ let ident = self.parse_ident().unwrap();
+ let msg = format!("add `struct` here to parse `{}` as a public struct", ident);
+ let mut err = self.struct_span_err(sp, "missing `struct` for struct definition");
+ err.span_suggestion_short(
+ sp,
+ &msg,
+ " struct ".into(),
+ Applicability::MaybeIncorrect, // speculative
+ );
+ return Err(err);
+ } else if self.look_ahead(1, |t| *t == token::OpenDelim(token::Paren)) {
+ let ident = self.parse_ident().unwrap();
+ self.bump(); // `(`
+ let kw_name = self.recover_first_param();
+ self.consume_block(token::Paren, ConsumeClosingDelim::Yes);
+ let (kw, kw_name, ambiguous) = if self.check(&token::RArrow) {
+ self.eat_to_tokens(&[&token::OpenDelim(token::Brace)]);
+ self.bump(); // `{`
+ ("fn", kw_name, false)
+ } else if self.check(&token::OpenDelim(token::Brace)) {
+ self.bump(); // `{`
+ ("fn", kw_name, false)
+ } else if self.check(&token::Colon) {
+ let kw = "struct";
+ (kw, kw, false)
+ } else {
+ ("fn` or `struct", "function or struct", true)
+ };
- // Verify whether we have encountered a struct or method definition where the user forgot to
- // add the `struct` or `fn` keyword after writing `pub`: `pub S {}`
- if vis.node.is_pub() && self.check_ident() && self.look_ahead(1, |t| *t != token::Not) {
- // Space between `pub` keyword and the identifier
- //
- // pub S {}
- // ^^^ `sp` points here
- let sp = self.prev_span.between(self.token.span);
- let full_sp = self.prev_span.to(self.token.span);
- let ident_sp = self.token.span;
- if self.look_ahead(1, |t| *t == token::OpenDelim(token::Brace)) {
- // possible public struct definition where `struct` was forgotten
- let ident = self.parse_ident().unwrap();
- let msg = format!("add `struct` here to parse `{}` as a public struct", ident);
- let mut err = self.struct_span_err(sp, "missing `struct` for struct definition");
+ let msg = format!("missing `{}` for {} definition", kw, kw_name);
+ let mut err = self.struct_span_err(sp, &msg);
+ if !ambiguous {
+ self.consume_block(token::Brace, ConsumeClosingDelim::Yes);
+ let suggestion =
+ format!("add `{}` here to parse `{}` as a public {}", kw, ident, kw_name);
err.span_suggestion_short(
sp,
- &msg,
- " struct ".into(),
- Applicability::MaybeIncorrect, // speculative
+ &suggestion,
+ format!(" {} ", kw),
+ Applicability::MachineApplicable,
);
- return Err(err);
- } else if self.look_ahead(1, |t| *t == token::OpenDelim(token::Paren)) {
- let ident = self.parse_ident().unwrap();
- self.bump(); // `(`
- let kw_name = self.recover_first_param();
- self.consume_block(token::Paren, ConsumeClosingDelim::Yes);
- let (kw, kw_name, ambiguous) = if self.check(&token::RArrow) {
- self.eat_to_tokens(&[&token::OpenDelim(token::Brace)]);
- self.bump(); // `{`
- ("fn", kw_name, false)
- } else if self.check(&token::OpenDelim(token::Brace)) {
- self.bump(); // `{`
- ("fn", kw_name, false)
- } else if self.check(&token::Colon) {
- let kw = "struct";
- (kw, kw, false)
- } else {
- ("fn` or `struct", "function or struct", true)
- };
-
- let msg = format!("missing `{}` for {} definition", kw, kw_name);
- let mut err = self.struct_span_err(sp, &msg);
- if !ambiguous {
- self.consume_block(token::Brace, ConsumeClosingDelim::Yes);
- let suggestion =
- format!("add `{}` here to parse `{}` as a public {}", kw, ident, kw_name);
- err.span_suggestion_short(
- sp,
- &suggestion,
- format!(" {} ", kw),
- Applicability::MachineApplicable,
+ } else {
+ if let Ok(snippet) = self.span_to_snippet(ident_sp) {
+ err.span_suggestion(
+ full_sp,
+ "if you meant to call a macro, try",
+ format!("{}!", snippet),
+ // this is the `ambiguous` conditional branch
+ Applicability::MaybeIncorrect,
);
} else {
- if let Ok(snippet) = self.span_to_snippet(ident_sp) {
- err.span_suggestion(
- full_sp,
- "if you meant to call a macro, try",
- format!("{}!", snippet),
- // this is the `ambiguous` conditional branch
- Applicability::MaybeIncorrect,
- );
- } else {
- err.help(
- "if you meant to call a macro, remove the `pub` \
+ err.help(
+ "if you meant to call a macro, remove the `pub` \
and add a trailing `!` after the identifier",
- );
- }
- }
- return Err(err);
- } else if self.look_ahead(1, |t| *t == token::Lt) {
- let ident = self.parse_ident().unwrap();
- self.eat_to_tokens(&[&token::Gt]);
- self.bump(); // `>`
- let (kw, kw_name, ambiguous) = if self.eat(&token::OpenDelim(token::Paren)) {
- ("fn", self.recover_first_param(), false)
- } else if self.check(&token::OpenDelim(token::Brace)) {
- ("struct", "struct", false)
- } else {
- ("fn` or `struct", "function or struct", true)
- };
- let msg = format!("missing `{}` for {} definition", kw, kw_name);
- let mut err = self.struct_span_err(sp, &msg);
- if !ambiguous {
- err.span_suggestion_short(
- sp,
- &format!("add `{}` here to parse `{}` as a public {}", kw, ident, kw_name),
- format!(" {} ", kw),
- Applicability::MachineApplicable,
);
}
- return Err(err);
}
+ return Err(err);
+ } else if self.look_ahead(1, |t| *t == token::Lt) {
+ let ident = self.parse_ident().unwrap();
+ self.eat_to_tokens(&[&token::Gt]);
+ self.bump(); // `>`
+ let (kw, kw_name, ambiguous) = if self.eat(&token::OpenDelim(token::Paren)) {
+ ("fn", self.recover_first_param(), false)
+ } else if self.check(&token::OpenDelim(token::Brace)) {
+ ("struct", "struct", false)
+ } else {
+ ("fn` or `struct", "function or struct", true)
+ };
+ let msg = format!("missing `{}` for {} definition", kw, kw_name);
+ let mut err = self.struct_span_err(sp, &msg);
+ if !ambiguous {
+ err.span_suggestion_short(
+ sp,
+ &format!("add `{}` here to parse `{}` as a public {}", kw, ident, kw_name),
+ format!(" {} ", kw),
+ Applicability::MachineApplicable,
+ );
+ }
+ return Err(err);
+ } else {
+ Ok(())
}
- self.parse_macro_use_or_failure(attrs, macros_allowed, attributes_allowed, lo, vis)
- }
-
- pub(super) fn mk_item_with_info(
- &self,
- attrs: Vec<Attribute>,
- lo: Span,
- vis: Visibility,
- info: ItemInfo,
- ) -> PResult<'a, Option<P<Item>>> {
- let (ident, item, extra_attrs) = info;
- let span = lo.to(self.prev_span);
- let attrs = Self::maybe_append(attrs, extra_attrs);
- Ok(Some(self.mk_item(span, ident, item, vis, attrs)))
- }
-
- fn maybe_append<T>(mut lhs: Vec<T>, mut rhs: Option<Vec<T>>) -> Vec<T> {
- if let Some(ref mut rhs) = rhs {
- lhs.append(rhs);
- }
- lhs
}
- /// This is the fall-through for parsing items.
- fn parse_macro_use_or_failure(
- &mut self,
- attrs: Vec<Attribute>,
- macros_allowed: bool,
- attributes_allowed: bool,
- lo: Span,
- visibility: Visibility,
- ) -> PResult<'a, Option<P<Item>>> {
- if macros_allowed
- && self.token.is_path_start()
- && !(self.is_async_fn() && self.token.span.rust_2015())
- {
- // MACRO INVOCATION ITEM
-
- let prev_span = self.prev_span;
- self.complain_if_pub_macro(&visibility.node, prev_span);
+ /// Parses an item macro, e.g., `item!();`.
+ fn parse_item_macro(&mut self, vis: &Visibility) -> PResult<'a, ItemInfo> {
+ self.complain_if_pub_macro(&vis.node, vis.span);
- // Item macro
- let path = self.parse_path(PathStyle::Mod)?;
- self.expect(&token::Not)?;
- let args = self.parse_mac_args()?;
- if args.need_semicolon() && !self.eat(&token::Semi) {
- self.report_invalid_macro_expansion_item();
- }
-
- let hi = self.prev_span;
- let mac = Mac { path, args, prior_type_ascription: self.last_type_ascription };
- let item =
- self.mk_item(lo.to(hi), Ident::invalid(), ItemKind::Mac(mac), visibility, attrs);
- return Ok(Some(item));
- }
-
- // FAILURE TO PARSE ITEM
- match visibility.node {
- VisibilityKind::Inherited => {}
- _ => return Err(self.struct_span_err(self.prev_span, "unmatched visibility `pub`")),
+ // Item macro
+ let path = self.parse_path(PathStyle::Mod)?;
+ self.expect(&token::Not)?;
+ let args = self.parse_mac_args()?;
+ if args.need_semicolon() && !self.eat(&token::Semi) {
+ self.report_invalid_macro_expansion_item();
}
- if !attributes_allowed && !attrs.is_empty() {
- self.expected_item_err(&attrs)?;
- }
- Ok(None)
+ let mac = Mac { path, args, prior_type_ascription: self.last_type_ascription };
+ Ok((Ident::invalid(), ItemKind::Mac(mac)))
}
/// Emits an expected-item-after-attributes error.
/// Parses an implementation item, `impl` keyword is already parsed.
///
- /// impl<'a, T> TYPE { /* impl items */ }
- /// impl<'a, T> TRAIT for TYPE { /* impl items */ }
- /// impl<'a, T> !TRAIT for TYPE { /* impl items */ }
- /// impl<'a, T> const TRAIT for TYPE { /* impl items */ }
+ /// ```
+ /// impl<'a, T> TYPE { /* impl items */ }
+ /// impl<'a, T> TRAIT for TYPE { /* impl items */ }
+ /// impl<'a, T> !TRAIT for TYPE { /* impl items */ }
+ /// impl<'a, T> const TRAIT for TYPE { /* impl items */ }
+ /// ```
///
/// We actually parse slightly more relaxed grammar for better error reporting and recovery.
- /// `impl` GENERICS `const`? `!`? TYPE `for`? (TYPE | `..`) (`where` PREDICATES)? `{` BODY `}`
- /// `impl` GENERICS `const`? `!`? TYPE (`where` PREDICATES)? `{` BODY `}`
+ /// ```
+ /// "impl" GENERICS "const"? "!"? TYPE "for"? (TYPE | "..") ("where" PREDICATES)? "{" BODY "}"
+ /// "impl" GENERICS "const"? "!"? TYPE ("where" PREDICATES)? "{" BODY "}"
+ /// ```
fn parse_item_impl(
&mut self,
+ attrs: &mut Vec<Attribute>,
unsafety: Unsafe,
defaultness: Defaultness,
) -> PResult<'a, ItemInfo> {
generics.where_clause = self.parse_where_clause()?;
- let (impl_items, attrs) = self.parse_impl_body()?;
+ let impl_items = self.parse_item_list(attrs, |p, at_end| p.parse_impl_item(at_end))?;
let item_kind = match ty_second {
Some(ty_second) => {
}
};
- Ok((Ident::invalid(), item_kind, Some(attrs)))
+ Ok((Ident::invalid(), item_kind))
}
- fn parse_impl_body(&mut self) -> PResult<'a, (Vec<P<AssocItem>>, Vec<Attribute>)> {
+ fn parse_item_list<T>(
+ &mut self,
+ attrs: &mut Vec<Attribute>,
+ mut parse_item: impl FnMut(&mut Parser<'a>, &mut bool) -> PResult<'a, T>,
+ ) -> PResult<'a, Vec<T>> {
self.expect(&token::OpenDelim(token::Brace))?;
- let attrs = self.parse_inner_attributes()?;
+ attrs.append(&mut self.parse_inner_attributes()?);
- let mut impl_items = Vec::new();
+ let mut items = Vec::new();
while !self.eat(&token::CloseDelim(token::Brace)) {
+ if self.recover_doc_comment_before_brace() {
+ continue;
+ }
let mut at_end = false;
- match self.parse_impl_item(&mut at_end) {
- Ok(impl_item) => impl_items.push(impl_item),
+ match parse_item(self, &mut at_end) {
+ Ok(item) => items.push(item),
Err(mut err) => {
err.emit();
if !at_end {
}
}
}
- Ok((impl_items, attrs))
+ Ok(items)
+ }
+
+ /// Recover on a doc comment before `}`.
+ fn recover_doc_comment_before_brace(&mut self) -> bool {
+ if let token::DocComment(_) = self.token.kind {
+ if self.look_ahead(1, |tok| tok == &token::CloseDelim(token::Brace)) {
+ struct_span_err!(
+ self.diagnostic(),
+ self.token.span,
+ E0584,
+ "found a documentation comment that doesn't document anything",
+ )
+ .span_label(self.token.span, "this doc comment doesn't document anything")
+ .help(
+ "doc comments must come before what they document, maybe a \
+ comment was intended with `//`?",
+ )
+ .emit();
+ self.bump();
+ return true;
+ }
+ }
+ false
}
/// Parses defaultness (i.e., `default` or nothing).
}
/// Parses `auto? trait Foo { ... }` or `trait Foo = Bar;`.
- fn parse_item_trait(&mut self, lo: Span, unsafety: Unsafe) -> PResult<'a, ItemInfo> {
+ fn parse_item_trait(
+ &mut self,
+ attrs: &mut Vec<Attribute>,
+ lo: Span,
+ unsafety: Unsafe,
+ ) -> PResult<'a, ItemInfo> {
// Parse optional `auto` prefix.
let is_auto = if self.eat_keyword(kw::Auto) { IsAuto::Yes } else { IsAuto::No };
self.sess.gated_spans.gate(sym::trait_alias, whole_span);
- Ok((ident, ItemKind::TraitAlias(tps, bounds), None))
+ Ok((ident, ItemKind::TraitAlias(tps, bounds)))
} else {
// It's a normal trait.
tps.where_clause = self.parse_where_clause()?;
- self.expect(&token::OpenDelim(token::Brace))?;
- let mut trait_items = vec![];
- while !self.eat(&token::CloseDelim(token::Brace)) {
- if let token::DocComment(_) = self.token.kind {
- if self.look_ahead(1, |tok| tok == &token::CloseDelim(token::Brace)) {
- struct_span_err!(
- self.diagnostic(),
- self.token.span,
- E0584,
- "found a documentation comment that doesn't document anything",
- )
- .help(
- "doc comments must come before what they document, maybe a \
- comment was intended with `//`?",
- )
- .emit();
- self.bump();
- continue;
- }
- }
- let mut at_end = false;
- match self.parse_trait_item(&mut at_end) {
- Ok(item) => trait_items.push(item),
- Err(mut e) => {
- e.emit();
- if !at_end {
- self.consume_block(token::Brace, ConsumeClosingDelim::Yes);
- break;
- }
- }
- }
- }
- Ok((ident, ItemKind::Trait(is_auto, unsafety, tps, bounds, trait_items), None))
+ let items = self.parse_item_list(attrs, |p, at_end| p.parse_trait_item(at_end))?;
+ Ok((ident, ItemKind::Trait(is_auto, unsafety, tps, bounds, items)))
}
}
fn parse_assoc_item(
&mut self,
at_end: &mut bool,
- is_name_required: fn(&token::Token) -> bool,
+ req_name: fn(&token::Token) -> bool,
) -> PResult<'a, P<AssocItem>> {
let attrs = self.parse_outer_attributes()?;
let mut unclosed_delims = vec![];
let (mut item, tokens) = self.collect_tokens(|this| {
- let item = this.parse_assoc_item_(at_end, attrs, is_name_required);
+ let item = this.parse_assoc_item_(at_end, attrs, req_name);
unclosed_delims.append(&mut this.unclosed_delims);
item
})?;
&mut self,
at_end: &mut bool,
mut attrs: Vec<Attribute>,
- is_name_required: fn(&token::Token) -> bool,
+ req_name: fn(&token::Token) -> bool,
) -> PResult<'a, AssocItem> {
let lo = self.token.span;
let vis = self.parse_visibility(FollowedByType::No)?;
let defaultness = self.parse_defaultness();
- let (name, kind, generics) = if self.eat_keyword(kw::Type) {
+
+ let (ident, kind, generics) = if self.eat_keyword(kw::Type) {
self.parse_assoc_ty()?
- } else if self.is_fn_front_matter() {
- let cfg = ParamCfg { is_name_required };
- let (ident, sig, generics, body) = self.parse_fn(at_end, &mut attrs, &cfg)?;
+ } else if self.check_fn_front_matter() {
+ let (ident, sig, generics, body) = self.parse_fn(at_end, &mut attrs, req_name)?;
(ident, AssocItemKind::Fn(sig, body), generics)
} else if let Some(mac) = self.parse_assoc_macro_invoc("associated", Some(&vis), at_end)? {
(Ident::invalid(), AssocItemKind::Macro(mac), Generics::default())
self.parse_assoc_const()?
};
- Ok(AssocItem {
- id: DUMMY_NODE_ID,
- span: lo.to(self.prev_span),
- ident: name,
- attrs,
- vis,
- defaultness,
- generics,
- kind,
- tokens: None,
- })
+ let span = lo.to(self.prev_span);
+ let id = DUMMY_NODE_ID;
+ Ok(AssocItem { id, span, ident, attrs, vis, defaultness, generics, kind, tokens: None })
}
/// This parses the grammar:
/// extern crate foo;
/// extern crate bar as foo;
/// ```
- fn parse_item_extern_crate(
- &mut self,
- lo: Span,
- visibility: Visibility,
- attrs: Vec<Attribute>,
- ) -> PResult<'a, P<Item>> {
+ fn parse_item_extern_crate(&mut self) -> PResult<'a, ItemInfo> {
// Accept `extern crate name-like-this` for better diagnostics
let orig_name = self.parse_crate_name_with_dashes()?;
let (item_name, orig_name) = if let Some(rename) = self.parse_rename()? {
(orig_name, None)
};
self.expect_semi()?;
-
- let span = lo.to(self.prev_span);
- Ok(self.mk_item(span, item_name, ItemKind::ExternCrate(orig_name), visibility, attrs))
+ Ok((item_name, ItemKind::ExternCrate(orig_name)))
}
fn parse_crate_name_with_dashes(&mut self) -> PResult<'a, ast::Ident> {
/// Parses `extern` for foreign ABIs modules.
///
- /// `extern` is expected to have been
- /// consumed before calling this method.
+ /// `extern` is expected to have been consumed before calling this method.
///
/// # Examples
///
/// extern "C" {}
/// extern {}
/// ```
- fn parse_item_foreign_mod(
- &mut self,
- lo: Span,
- abi: Option<StrLit>,
- visibility: Visibility,
- mut attrs: Vec<Attribute>,
- ) -> PResult<'a, P<Item>> {
- self.expect(&token::OpenDelim(token::Brace))?;
-
- attrs.extend(self.parse_inner_attributes()?);
-
- let mut foreign_items = vec![];
- while !self.eat(&token::CloseDelim(token::Brace)) {
- foreign_items.push(self.parse_foreign_item()?);
- }
-
- let prev_span = self.prev_span;
- let m = ast::ForeignMod { abi, items: foreign_items };
- let invalid = Ident::invalid();
- Ok(self.mk_item(lo.to(prev_span), invalid, ItemKind::ForeignMod(m), visibility, attrs))
+ fn parse_item_foreign_mod(&mut self, attrs: &mut Vec<Attribute>) -> PResult<'a, ItemInfo> {
+ let abi = self.parse_abi(); // ABI?
+ let items = self.parse_item_list(attrs, |p, at_end| p.parse_foreign_item(at_end))?;
+ let module = ast::ForeignMod { abi, items };
+ Ok((Ident::invalid(), ItemKind::ForeignMod(module)))
}
- /// Parses a foreign item.
- pub fn parse_foreign_item(&mut self) -> PResult<'a, P<ForeignItem>> {
+ /// Parses a foreign item (one in an `extern { ... }` block).
+ pub fn parse_foreign_item(&mut self, at_end: &mut bool) -> PResult<'a, P<ForeignItem>> {
maybe_whole!(self, NtForeignItem, |ni| ni);
let mut attrs = self.parse_outer_attributes()?;
let lo = self.token.span;
let vis = self.parse_visibility(FollowedByType::No)?;
- if self.check_keyword(kw::Type) {
+ let (ident, kind) = if self.check_keyword(kw::Type) {
// FOREIGN TYPE ITEM
- self.parse_item_foreign_type(vis, lo, attrs)
- } else if self.is_fn_front_matter() {
+ self.parse_item_foreign_type()?
+ } else if self.check_fn_front_matter() {
// FOREIGN FUNCTION ITEM
- let (ident, sig, generics, body) =
- self.parse_fn(&mut false, &mut attrs, &ParamCfg::FREE)?;
- let kind = ForeignItemKind::Fn(sig, generics, body);
- let span = lo.to(self.prev_span);
- Ok(P(ast::ForeignItem {
- ident,
- attrs,
- kind,
- id: DUMMY_NODE_ID,
- span,
- vis,
- tokens: None,
- }))
+ let (ident, sig, generics, body) = self.parse_fn(at_end, &mut attrs, |_| true)?;
+ (ident, ForeignItemKind::Fn(sig, generics, body))
} else if self.is_static_global() {
// FOREIGN STATIC ITEM
self.bump(); // `static`
- self.parse_item_foreign_static(vis, lo, attrs)
+ self.parse_item_foreign_static()?
} else if self.token.is_keyword(kw::Const) {
// Treat `const` as `static` for error recovery, but don't add it to expected tokens.
self.bump(); // `const`
Applicability::MachineApplicable,
)
.emit();
- self.parse_item_foreign_static(vis, lo, attrs)
- } else if let Some(mac) = self.parse_assoc_macro_invoc("extern", Some(&vis), &mut false)? {
- let kind = ForeignItemKind::Macro(mac);
- let span = lo.to(self.prev_span);
- let ident = Ident::invalid();
- Ok(P(ForeignItem { ident, span, id: DUMMY_NODE_ID, attrs, vis, kind, tokens: None }))
+ self.parse_item_foreign_static()?
+ } else if let Some(mac) = self.parse_assoc_macro_invoc("extern", Some(&vis), at_end)? {
+ (Ident::invalid(), ForeignItemKind::Macro(mac))
} else {
if !attrs.is_empty() {
self.expected_item_err(&attrs)?;
}
- self.unexpected()
- }
+ self.unexpected()?
+ };
+
+ let span = lo.to(self.prev_span);
+ Ok(P(ast::ForeignItem { ident, attrs, kind, id: DUMMY_NODE_ID, span, vis, tokens: None }))
}
/// Parses a static item from a foreign module.
/// Assumes that the `static` keyword is already parsed.
- fn parse_item_foreign_static(
- &mut self,
- vis: ast::Visibility,
- lo: Span,
- attrs: Vec<Attribute>,
- ) -> PResult<'a, P<ForeignItem>> {
+ fn parse_item_foreign_static(&mut self) -> PResult<'a, (Ident, ForeignItemKind)> {
let mutbl = self.parse_mutability();
let ident = self.parse_ident()?;
self.expect(&token::Colon)?;
let ty = self.parse_ty()?;
- let hi = self.token.span;
self.expect_semi()?;
- Ok(P(ForeignItem {
- ident,
- attrs,
- kind: ForeignItemKind::Static(ty, mutbl),
- id: DUMMY_NODE_ID,
- span: lo.to(hi),
- vis,
- tokens: None,
- }))
+ Ok((ident, ForeignItemKind::Static(ty, mutbl)))
}
/// Parses a type from a foreign module.
- fn parse_item_foreign_type(
- &mut self,
- vis: ast::Visibility,
- lo: Span,
- attrs: Vec<Attribute>,
- ) -> PResult<'a, P<ForeignItem>> {
+ fn parse_item_foreign_type(&mut self) -> PResult<'a, (Ident, ForeignItemKind)> {
self.expect_keyword(kw::Type)?;
-
let ident = self.parse_ident()?;
- let hi = self.token.span;
self.expect_semi()?;
- Ok(P(ast::ForeignItem {
- ident,
- attrs,
- kind: ForeignItemKind::Ty,
- id: DUMMY_NODE_ID,
- span: lo.to(hi),
- vis,
- tokens: None,
- }))
+ Ok((ident, ForeignItemKind::Ty))
}
fn is_static_global(&mut self) -> bool {
Some(m) => ItemKind::Static(ty, m, e),
None => ItemKind::Const(ty, e),
};
- Ok((id, item, None))
+ Ok((id, item))
}
/// We were supposed to parse `:` but instead, we're already at `=`.
let enum_definition =
EnumDef { variants: variants.into_iter().filter_map(|v| v).collect() };
- Ok((id, ItemKind::Enum(enum_definition, generics), None))
+ Ok((id, ItemKind::Enum(enum_definition, generics)))
}
fn parse_enum_variant(&mut self) -> PResult<'a, Option<Variant>> {
return Err(err);
};
- Ok((class_name, ItemKind::Struct(vdata, generics), None))
+ Ok((class_name, ItemKind::Struct(vdata, generics)))
}
/// Parses `union Foo { ... }`.
return Err(err);
};
- Ok((class_name, ItemKind::Union(vdata, generics), None))
+ Ok((class_name, ItemKind::Union(vdata, generics)))
}
pub(super) fn is_union_item(&self) -> bool {
})
}
- pub(super) fn eat_macro_def(
- &mut self,
- attrs: &[Attribute],
- vis: &Visibility,
- lo: Span,
- ) -> PResult<'a, Option<P<Item>>> {
- let (ident, def) = if self.eat_keyword(kw::Macro) {
- let ident = self.parse_ident()?;
+ /// Parses a declarative macro 2.0 definition.
+ /// The `macro` keyword has already been parsed.
+ fn parse_item_decl_macro(&mut self, lo: Span) -> PResult<'a, ItemInfo> {
+ let ident = self.parse_ident()?;
+ let body = if self.check(&token::OpenDelim(token::Brace)) {
+ self.parse_mac_args()?
+ } else if self.check(&token::OpenDelim(token::Paren)) {
+ let params = self.parse_token_tree();
+ let pspan = params.span();
let body = if self.check(&token::OpenDelim(token::Brace)) {
- self.parse_mac_args()?
- } else if self.check(&token::OpenDelim(token::Paren)) {
- let params = self.parse_token_tree();
- let pspan = params.span();
- let body = if self.check(&token::OpenDelim(token::Brace)) {
- self.parse_token_tree()
- } else {
- return self.unexpected();
- };
- let bspan = body.span();
- let tokens = TokenStream::new(vec![
- params.into(),
- TokenTree::token(token::FatArrow, pspan.between(bspan)).into(),
- body.into(),
- ]);
- let dspan = DelimSpan::from_pair(pspan.shrink_to_lo(), bspan.shrink_to_hi());
- P(MacArgs::Delimited(dspan, MacDelimiter::Brace, tokens))
+ self.parse_token_tree()
} else {
return self.unexpected();
};
+ let bspan = body.span();
+ let tokens = TokenStream::new(vec![
+ params.into(),
+ TokenTree::token(token::FatArrow, pspan.between(bspan)).into(),
+ body.into(),
+ ]);
+ let dspan = DelimSpan::from_pair(pspan.shrink_to_lo(), bspan.shrink_to_hi());
+ P(MacArgs::Delimited(dspan, MacDelimiter::Brace, tokens))
+ } else {
+ return self.unexpected();
+ };
+
+ self.sess.gated_spans.gate(sym::decl_macro, lo.to(self.prev_span));
+ Ok((ident, ItemKind::MacroDef(ast::MacroDef { body, legacy: false })))
+ }
- (ident, ast::MacroDef { body, legacy: false })
- } else if self.check_keyword(sym::macro_rules)
+ /// Is this unambiguously the start of a `macro_rules! foo` item defnition?
+ fn is_macro_rules_item(&mut self) -> bool {
+ self.check_keyword(sym::macro_rules)
&& self.look_ahead(1, |t| *t == token::Not)
&& self.look_ahead(2, |t| t.is_ident())
- {
- let prev_span = self.prev_span;
- self.complain_if_pub_macro(&vis.node, prev_span);
- self.bump();
- self.bump();
-
- let ident = self.parse_ident()?;
- let body = self.parse_mac_args()?;
- if body.need_semicolon() && !self.eat(&token::Semi) {
- self.report_invalid_macro_expansion_item();
- }
-
- (ident, ast::MacroDef { body, legacy: true })
- } else {
- return Ok(None);
- };
+ }
- let span = lo.to(self.prev_span);
+ /// Parses a legacy `macro_rules! foo { ... }` declarative macro.
+ fn parse_item_macro_rules(&mut self, vis: &Visibility) -> PResult<'a, ItemInfo> {
+ self.complain_if_pub_macro(&vis.node, vis.span);
+ self.expect_keyword(sym::macro_rules)?; // `macro_rules`
+ self.expect(&token::Not)?; // `!`
- if !def.legacy {
- self.sess.gated_spans.gate(sym::decl_macro, span);
+ let ident = self.parse_ident()?;
+ let body = self.parse_mac_args()?;
+ if body.need_semicolon() && !self.eat(&token::Semi) {
+ self.report_invalid_macro_expansion_item();
}
- Ok(Some(self.mk_item(span, ident, ItemKind::MacroDef(def), vis.clone(), attrs.to_vec())))
+ Ok((ident, ItemKind::MacroDef(ast::MacroDef { body, legacy: true })))
+ }
+
+ pub(super) fn eat_macro_def(
+ &mut self,
+ attrs: &[Attribute],
+ vis: &Visibility,
+ lo: Span,
+ ) -> PResult<'a, Option<P<Item>>> {
+ let (ident, kind) = if self.eat_keyword(kw::Macro) {
+ self.parse_item_decl_macro(lo)?
+ } else if self.is_macro_rules_item() {
+ self.parse_item_macro_rules(vis)?
+ } else {
+ return Ok(None);
+ };
+ Ok(Some(self.mk_item(lo.to(self.prev_span), ident, kind, vis.clone(), attrs.to_vec())))
}
fn complain_if_pub_macro(&self, vis: &VisibilityKind, sp: Span) {
}
/// The parsing configuration used to parse a parameter list (see `parse_fn_params`).
-pub(super) struct ParamCfg {
- /// `is_name_required` decides if, per-parameter,
- /// the parameter must have a pattern or just a type.
- pub is_name_required: fn(&token::Token) -> bool,
-}
-
-impl ParamCfg {
- /// Configuration for a free function in the sense that it is not associated.
- const FREE: Self = ParamCfg { is_name_required: |_| true };
-}
+///
+/// The function decides if, per-parameter `p`, `p` must have a pattern or just a type.
+type ReqName = fn(&token::Token) -> bool;
/// Parsing of functions and methods.
impl<'a> Parser<'a> {
&mut self,
at_end: &mut bool,
attrs: &mut Vec<Attribute>,
- cfg: &ParamCfg,
+ req_name: ReqName,
) -> PResult<'a, (Ident, FnSig, Generics, Option<P<Block>>)> {
let header = self.parse_fn_front_matter()?; // `const ... fn`
let ident = self.parse_ident()?; // `foo`
let mut generics = self.parse_generics()?; // `<'a, T, ...>`
- let decl = self.parse_fn_decl(cfg, AllowPlus::Yes)?; // `(p: u8, ...)`
+ let decl = self.parse_fn_decl(req_name, AllowPlus::Yes)?; // `(p: u8, ...)`
generics.where_clause = self.parse_where_clause()?; // `where T: Ord`
let body = self.parse_fn_body(at_end, attrs)?; // `;` or `{ ... }`.
Ok((ident, FnSig { header, decl }, generics, body))
}
/// Is the current token the start of an `FnHeader` / not a valid parse?
- fn is_fn_front_matter(&mut self) -> bool {
+ fn check_fn_front_matter(&mut self) -> bool {
// We use an over-approximation here.
// `const const`, `fn const` won't parse, but we're not stepping over other syntax either.
const QUALS: [Symbol; 4] = [kw::Const, kw::Async, kw::Unsafe, kw::Extern];
/// Parses the parameter list and result type of a function declaration.
pub(super) fn parse_fn_decl(
&mut self,
- cfg: &ParamCfg,
+ req_name: ReqName,
ret_allow_plus: AllowPlus,
) -> PResult<'a, P<FnDecl>> {
Ok(P(FnDecl {
- inputs: self.parse_fn_params(cfg)?,
+ inputs: self.parse_fn_params(req_name)?,
output: self.parse_ret_ty(ret_allow_plus, RecoverQPath::Yes)?,
}))
}
/// Parses the parameter list of a function, including the `(` and `)` delimiters.
- fn parse_fn_params(&mut self, cfg: &ParamCfg) -> PResult<'a, Vec<Param>> {
+ fn parse_fn_params(&mut self, req_name: ReqName) -> PResult<'a, Vec<Param>> {
let mut first_param = true;
// Parse the arguments, starting out with `self` being allowed...
let (mut params, _) = self.parse_paren_comma_seq(|p| {
- let param = p.parse_param_general(&cfg, first_param).or_else(|mut e| {
+ let param = p.parse_param_general(req_name, first_param).or_else(|mut e| {
e.emit();
let lo = p.prev_span;
// Skip every token until next possible arg or end.
/// Parses a single function parameter.
///
/// - `self` is syntactically allowed when `first_param` holds.
- fn parse_param_general(&mut self, cfg: &ParamCfg, first_param: bool) -> PResult<'a, Param> {
+ fn parse_param_general(&mut self, req_name: ReqName, first_param: bool) -> PResult<'a, Param> {
let lo = self.token.span;
let attrs = self.parse_outer_attributes()?;
let is_name_required = match self.token.kind {
token::DotDotDot => false,
- _ => (cfg.is_name_required)(&self.token),
+ _ => req_name(&self.token),
};
let (pat, ty) = if is_name_required || self.is_named_param() {
debug!("parse_param_general parse_pat (is_name_required:{})", is_name_required);