---

yaml
---
r: 22516
b: refs/heads/master
c: 78d11b8
h: refs/heads/master
v: v3

Author: lkuper

[refs]

@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 5a63b2100ed69b14e929671430101cc786f41ea7
+refs/heads/master: 78d11b899b6ec4d2ed8c18499c1213fecd3c6424
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: cd6f24f9d14ac90d167386a56e7a6ac1f0318195
 refs/heads/try: ffbe0e0e00374358b789b0037bcb3a577cd218be

trunk/src/libsyntax/parse/parser.rs

@@ -279,41 +279,46 @@ class parser {
         do self.parse_unspanned_seq(token::LBRACE, token::RBRACE,
                                     seq_sep_none()) |p| {
             let attrs = p.parse_outer_attributes();
-            let flo = p.span.lo;
+            let lo = p.span.lo;
             let pur = p.parse_fn_purity();
+            // NB: at the moment, trait methods are public by default; this
+            // could change.
+            let vis = p.parse_visibility(public);
             let ident = p.parse_method_name();
             let tps = p.parse_ty_params();
-            let d = p.parse_ty_fn_decl(pur), fhi = p.last_span.hi;
-            #debug["parse_trait_methods(): trait method ends in %s",
-                   token_to_str(self.reader, self.token)];
-            alt self.token {
+            let d = p.parse_ty_fn_decl(pur);
+            let hi = p.last_span.hi;
+            #debug["parse_trait_methods(): trait method signature ends in \
+                    `%s`",
+                   token_to_str(p.reader, p.token)];
+            alt p.token {
               token::SEMI {
-                self.bump();
+                p.bump();
+                #debug["parse_trait_methods(): parsing required method"];
+                // NB: at the moment, visibility annotations on required
+                // methods are ignored; this could change.
                 required({ident: ident, attrs: attrs,
                           decl: {purity: pur with d}, tps: tps,
-                          span: mk_sp(flo, fhi)})
+                          span: mk_sp(lo, hi)})
               }
               token::LBRACE {
-                self.bump();
+                #debug["parse_trait_methods(): parsing provided method"];
                 let (inner_attrs, body) =
-                    self.parse_inner_attrs_and_block(true);
+                    p.parse_inner_attrs_and_block(true);
                 let attrs = vec::append(attrs, inner_attrs);
-                self.eat(token::RBRACE);
                 provided(@{ident: ident,
                            attrs: attrs,
                            tps: tps,
                            decl: d,
                            body: body,
-                           id: self.get_id(),
-                           span: mk_sp(flo, fhi),
-                           self_id: self.get_id(),
-                           // Provided traits methods always public for now
-                           vis: public})
+                           id: p.get_id(),
+                           span: mk_sp(lo, hi),
+                           self_id: p.get_id(),
+                           vis: vis})
               }
 
-              _ { self.fatal("expected `;` or `}` \
-                              but found `"
-                             + token_to_str(self.reader, self.token) + "`");
+              _ { p.fatal("expected `;` or `}` but found `" +
+                          token_to_str(p.reader, p.token) + "`");
                 }
             }
         }