Skip to content

Commit

Permalink
Fix rebase fallout
Browse files Browse the repository at this point in the history
  • Loading branch information
sfackler committed Sep 24, 2014
1 parent 65cca7c commit dcdbdc1
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 14 deletions.
2 changes: 1 addition & 1 deletion src/libsyntax/ext/base.rs
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,7 @@ fn initial_syntax_expander_table() -> SyntaxEnv {
builtin_normal_expander(
ext::cfg::expand_cfg));
syntax_expanders.insert(intern("cfg_attr"),
ItemModifier(ext::cfg_attr::expand));
Modifier(box ext::cfg_attr::expand));
syntax_expanders.insert(intern("trace_macros"),
builtin_normal_expander(
ext::trace_macros::expand_trace_macros));
Expand Down
22 changes: 10 additions & 12 deletions src/libsyntax/ext/cfg_attr.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,45 +8,43 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

use std::gc::{Gc, GC};

use ast;
use attr;
use codemap::Span;
use ext::base::ExtCtxt;
use ext::build::AstBuilder;
use ptr::P;

pub fn expand(cx: &mut ExtCtxt, sp: Span, mi: Gc<ast::MetaItem>, it: Gc<ast::Item>)
-> Gc<ast::Item> {
pub fn expand(cx: &mut ExtCtxt, sp: Span, mi: &ast::MetaItem, it: P<ast::Item>) -> P<ast::Item> {
let (cfg, attr) = match mi.node {
ast::MetaList(_, ref mis) if mis.len() == 2 => (mis[0], mis[1]),
ast::MetaList(_, ref mis) if mis.len() == 2 => (&mis[0], &mis[1]),
_ => {
cx.span_err(sp, "expected `#[cfg_attr(<cfg pattern>, <attr>)]`");
return it;
}
};

let mut out = (*it).clone();
if cfg_matches(cx, cfg) {
out.attrs.push(cx.attribute(attr.span, attr));
if cfg_matches(cx, &**cfg) {
out.attrs.push(cx.attribute(attr.span, attr.clone()));
}

box(GC) out
P(out)
}

fn cfg_matches(cx: &mut ExtCtxt, cfg: Gc<ast::MetaItem>) -> bool {
fn cfg_matches(cx: &mut ExtCtxt, cfg: &ast::MetaItem) -> bool {
match cfg.node {
ast::MetaList(ref pred, ref mis) if pred.get() == "any" =>
mis.iter().any(|mi| cfg_matches(cx, *mi)),
mis.iter().any(|mi| cfg_matches(cx, &**mi)),
ast::MetaList(ref pred, ref mis) if pred.get() == "all" =>
mis.iter().all(|mi| cfg_matches(cx, *mi)),
mis.iter().all(|mi| cfg_matches(cx, &**mi)),
ast::MetaList(ref pred, ref mis) if pred.get() == "not" => {
if mis.len() != 1 {
cx.span_err(cfg.span, format!("expected 1 value, got {}",
mis.len()).as_slice());
return false;
}
!cfg_matches(cx, mis[0])
!cfg_matches(cx, &*mis[0])
}
ast::MetaList(ref pred, _) => {
cx.span_err(cfg.span,
Expand Down
2 changes: 1 addition & 1 deletion src/libsyntax/test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,7 @@ fn is_ignored(cx: &TestCtxt, i: &ast::Item) -> bool {
attr.check_name("ignore") && match attr.meta_item_list() {
Some(ref cfgs) => {
if cfgs.iter().any(|cfg| cfg.check_name("cfg")) {
cx.sess.span_warn(attr.span,
cx.span_diagnostic.span_warn(attr.span,
"The use of cfg filters in #[ignore] is \
deprecated. Use #[cfg_attr(<cfg pattern>, \
ignore)] instead.");
Expand Down

17 comments on commit dcdbdc1

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@dcdbdc1

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/cfg-attr = dcdbdc1 into auto

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/cfg-attr = dcdbdc1 merged ok, testing candidate = e8256274

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@dcdbdc1

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/cfg-attr = dcdbdc1 into auto

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/cfg-attr = dcdbdc1 merged ok, testing candidate = 57b4b217

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@dcdbdc1

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/cfg-attr = dcdbdc1 into auto

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/cfg-attr = dcdbdc1 merged ok, testing candidate = 089409da

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 26, 2014

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@dcdbdc1

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/cfg-attr = dcdbdc1 into auto

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/cfg-attr = dcdbdc1 merged ok, testing candidate = 43d7d7c

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on dcdbdc1 Sep 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 43d7d7c

Please sign in to comment.