auto merge of #9525 : klutzy/rust/obsolete-span-fix, r=alexcrichton
This commit is contained in:
commit
eb3ebb7ebd
1 changed files with 9 additions and 6 deletions
|
@ -922,6 +922,7 @@ impl Parser {
|
||||||
let attrs = p.parse_outer_attributes();
|
let attrs = p.parse_outer_attributes();
|
||||||
let lo = p.span.lo;
|
let lo = p.span.lo;
|
||||||
|
|
||||||
|
let vis_span = *self.span;
|
||||||
let vis = p.parse_visibility();
|
let vis = p.parse_visibility();
|
||||||
let pur = p.parse_fn_purity();
|
let pur = p.parse_fn_purity();
|
||||||
// NB: at the moment, trait methods are public by default; this
|
// NB: at the moment, trait methods are public by default; this
|
||||||
|
@ -947,7 +948,7 @@ impl Parser {
|
||||||
// NB: at the moment, visibility annotations on required
|
// NB: at the moment, visibility annotations on required
|
||||||
// methods are ignored; this could change.
|
// methods are ignored; this could change.
|
||||||
if vis != ast::inherited {
|
if vis != ast::inherited {
|
||||||
self.obsolete(*self.last_span,
|
self.obsolete(vis_span,
|
||||||
ObsoleteTraitFuncVisibility);
|
ObsoleteTraitFuncVisibility);
|
||||||
}
|
}
|
||||||
required(TypeMethod {
|
required(TypeMethod {
|
||||||
|
@ -1213,14 +1214,16 @@ impl Parser {
|
||||||
// parse an optional, obsolete argument mode.
|
// parse an optional, obsolete argument mode.
|
||||||
pub fn parse_arg_mode(&self) {
|
pub fn parse_arg_mode(&self) {
|
||||||
if self.eat(&token::BINOP(token::MINUS)) {
|
if self.eat(&token::BINOP(token::MINUS)) {
|
||||||
self.obsolete(*self.span, ObsoleteMode);
|
self.obsolete(*self.last_span, ObsoleteMode);
|
||||||
} else if self.eat(&token::ANDAND) {
|
} else if self.eat(&token::ANDAND) {
|
||||||
self.obsolete(*self.span, ObsoleteMode);
|
self.obsolete(*self.last_span, ObsoleteMode);
|
||||||
} else if self.eat(&token::BINOP(token::PLUS)) {
|
} else if self.eat(&token::BINOP(token::PLUS)) {
|
||||||
|
let lo = self.last_span.lo;
|
||||||
if self.eat(&token::BINOP(token::PLUS)) {
|
if self.eat(&token::BINOP(token::PLUS)) {
|
||||||
self.obsolete(*self.span, ObsoleteMode);
|
let hi = self.last_span.hi;
|
||||||
|
self.obsolete(mk_sp(lo, hi), ObsoleteMode);
|
||||||
} else {
|
} else {
|
||||||
self.obsolete(*self.span, ObsoleteMode);
|
self.obsolete(*self.last_span, ObsoleteMode);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// Ignore.
|
// Ignore.
|
||||||
|
@ -3865,7 +3868,7 @@ impl Parser {
|
||||||
|
|
||||||
let mut meths = ~[];
|
let mut meths = ~[];
|
||||||
if self.eat(&token::SEMI) {
|
if self.eat(&token::SEMI) {
|
||||||
self.obsolete(*self.span, ObsoleteEmptyImpl);
|
self.obsolete(*self.last_span, ObsoleteEmptyImpl);
|
||||||
} else {
|
} else {
|
||||||
self.expect(&token::LBRACE);
|
self.expect(&token::LBRACE);
|
||||||
while !self.eat(&token::RBRACE) {
|
while !self.eat(&token::RBRACE) {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue