rust/src/libsyntax/parse/parser.rs, branch 1.33.0

Rollup merge of #57585 - estebank:trailing-semicolon, r=petrochenkov

2019-01-14T19:31:58+00:00

Recover from item trailing semicolon CC https://github.com/rust-lang/rfcs/pull/2479 r? @petrochenkov

Rollup merge of #57572 - Centril:unaccept-extern-in-path, r=petrochenkov

2019-01-14T19:31:57+00:00

Unaccept `extern_in_paths` Based on completed fcp-close in https://github.com/rust-lang/rust/issues/55600, this removes `extern_in_path` (e.g. `extern::foo::bar`) from the language. The changes are primarily reversing https://github.com/rust-lang/rust/commit/32db83b16e06cb5cca72d0e6a648a8008eda0fac. Closes https://github.com/rust-lang/rust/issues/55600 r? @petrochenkov

Rollup merge of #57540 - estebank:eval-more, r=petrochenkov

2019-01-14T19:31:55+00:00

Modify some parser diagnostics to continue evaluating beyond the parser Continue evaluating further errors after parser errors on: - trailing type argument attribute - lifetime in incorrect location - incorrect binary literal - missing `for` in `impl Trait for Foo` - type argument in `where` clause - incorrect float literal - incorrect `..` in pattern - associated types - incorrect discriminator value variant error and others. All of these were found by making `continue-parse-after-error` `true` by default to identify errors that would need few changes. There are now only a handful of errors that have any change with `continue-parse-after-error` enabled. These changes make it so `rust` _won't_ stop evaluation after finishing parsing, enabling type checking errors to be displayed on the existing code without having to fix the parse errors. Each commit has an individual diagnostic change with their corresponding tests. CC #48724.

Recover from item trailing semicolon

2019-01-14T01:29:32+00:00

Rollup merge of #57004 - nnethercote:TS-change-Stream, r=petrochenkov

2019-01-13T16:21:39+00:00

Make `TokenStream` less recursive. `TokenStream` is currently recursive in *two* ways: - the `TokenTree` variant contains a `ThinTokenStream`, which can contain a `TokenStream`; - the `TokenStream` variant contains a `Vec`. The latter is not necessary and causes significant complexity. This commit replaces it with the simpler `Vec<(TokenTree, IsJoint)>`. This reduces complexity significantly. In particular, `StreamCursor` is eliminated, and `Cursor` becomes much simpler, consisting now of just a `TokenStream` and an index. The commit also removes the `Extend` impl for `TokenStream`, because it is only used in tests. (The commit also removes those tests.) Overall, the commit reduces the number of lines of code by almost 200.

remove extern_in_paths.

2019-01-13T13:18:00+00:00

Suggest correct location for lifetime parameters in use

2019-01-13T03:25:03+00:00

Tweak incorrect discriminator value variant error

2019-01-12T07:12:29+00:00

Small tweaks to parser errors

2019-01-12T06:04:54+00:00

Tweak type argument after assoc type error

2019-01-12T05:45:24+00:00