Add [ to the follow set for ty #31135

Activity

1. nikomatsakis

   added

   A-macrosArea: All kinds of macros (custom derive, macro_rules!, proc macros, ..)Area: All kinds of macros (custom derive, macro_rules!, proc macros, ..)

   T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.

   on Jan 22, 2016
2. nikomatsakis

   assigned

   pnkfelix

   on Jan 22, 2016
3. durka commented on Jan 23, 2016

   

   durka

   on Jan 23, 2016

   Contributor

   More actions

   So according to the RFC document, FOLLOW(path) = FOLLOW(ty). Does that mean we are adding [ to FOLLOW(path) as well? (My assumption is no.)
4. bors

   added a commit that references this issue on Jan 23, 2016

   Auto merge of #31152 - durka:ty-follow-bracket, r=pnkfelix

   ...

   d5fcdcc
5. nikomatsakis commented on Jan 25, 2016

   

   nikomatsakis

   on Jan 25, 2016

   ContributorAuthor

   More actions

   Seems like yes. After all, a type can be just a path, so if we added a
   [ suffix on paths, it would effectively add such a suffix to types
   as well.

   On Sat, Jan 23, 2016 at 09:20:33AM -0800, Alex Burka wrote:

   So according to the RFC document, FOLLOW(path) = FOLLOW(ty). Does that mean we are adding [ to FOLLOW(path) as well?

   ---

   Reply to this email directly or view it on GitHub:
   #31135 (comment)
6. Manishearth

   added a commit that references this issue on Jan 26, 2016

   Rollup merge of rust-lang#31152 - durka:ty-follow-bracket, r=pnkfelix

   ...

   b46bd2f
7. bors

   closed this as completedin e1e0de8on Jan 26, 2016
8. pnkfelix

   added a commit that references this issue on Feb 5, 2016

   add `[` to FOLLOW(ty) and FOLLOW(path)

   ...

   This issue will close once commit 46049ee is merged into the 'master' branch.

   46049ee