Skip to content

Commit

Permalink
Auto merge of #39145 - nrc:opt-cfg-mod, r=@jseyfried
Browse files Browse the repository at this point in the history
Add an option to the parser so cfg'ed out modules can still be parsed

r? @jseyfried

cc @dtolnay, @erickt it would be great if we could get this picked up into Syntex asap - it fixes a pretty nasty bug in Rustfmt.
  • Loading branch information
bors committed Jan 24, 2017
2 parents 0ba6038 + 44180c8 commit ae47248
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/libsyntax/parse/parser.rs
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,8 @@ pub struct Parser<'a> {
pub expected_tokens: Vec<TokenType>,
pub tts: Vec<(TokenTree, usize)>,
pub desugar_doc_comments: bool,
/// Whether we should configure out of line modules as we parse.
pub cfg_mods: bool,
}

#[derive(PartialEq, Eq, Clone)]
Expand Down Expand Up @@ -273,6 +275,7 @@ impl<'a> Parser<'a> {
expected_tokens: Vec::new(),
tts: if tt.len() > 0 { vec![(tt, 0)] } else { Vec::new() },
desugar_doc_comments: desugar_doc_comments,
cfg_mods: true,
};

let tok = parser.next_tok();
Expand Down Expand Up @@ -5210,7 +5213,7 @@ impl<'a> Parser<'a> {
features: None, // don't perform gated feature checking
};
let outer_attrs = strip_unconfigured.process_cfg_attrs(outer_attrs.to_owned());
(strip_unconfigured.in_cfg(&outer_attrs), outer_attrs)
(!self.cfg_mods || strip_unconfigured.in_cfg(&outer_attrs), outer_attrs)
};

let id_span = self.span;
Expand Down Expand Up @@ -5396,6 +5399,7 @@ impl<'a> Parser<'a> {

let mut p0 =
new_sub_parser_from_file(self.sess, &path, directory_ownership, Some(name), id_sp);
p0.cfg_mods = self.cfg_mods;
let mod_inner_lo = p0.span.lo;
let mod_attrs = p0.parse_inner_attributes()?;
let m0 = p0.parse_mod_items(&token::Eof, mod_inner_lo)?;
Expand Down

0 comments on commit ae47248

Please sign in to comment.