Fix empty lifetime list or one with trailing comma being rejected

Fixes #37733

diff --git a/src/libsyntax/parse/parser.rs b/src/libsyntax/parse/parser.rs
index b80aa667be6a190064f4b2aa3c2a8bda4b97fa59..064c93c2866e9f8af6cc7ad30d624bb9f2b948a8 100644 (file)
--- a/src/libsyntax/parse/parser.rs
+++ b/src/libsyntax/parse/parser.rs
@@ -1943,12 +1943,12 @@ pub fn parse_lifetime_defs(&mut self,
                     if let Some(recv) = followed_by_ty_params {
                         assert!(recv.is_empty());
                         *recv = attrs;
-                    } else {
+                        debug!("parse_lifetime_defs ret {:?}", res);
+                        return Ok(res);
+                    } else if !attrs.is_empty() {
                         let msg = "trailing attribute after lifetime parameters";
                         return Err(self.fatal(msg));
                     }
-                    debug!("parse_lifetime_defs ret {:?}", res);
-                    return Ok(res);
                 }
             }
 

diff --git a/src/test/run-pass/issue-37733.rs b/src/test/run-pass/issue-37733.rs
new file mode 100644 (file)
index 0000000..358b932
--- /dev/null
+++ b/src/test/run-pass/issue-37733.rs
@@ -0,0 +1,15 @@
+// Copyright 2016 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+type A = for<> fn();
+
+type B = for<'a,> fn();
+
+pub fn main() {}