I agree with John; we have three cases here:

- do we require a name? if so, it's a grouping paren
- do we have a (possibly empty) list of parameters? if so, it's not a grouping 
paren
- otherwise, it's a grouping paren

It doesn't make sense to combine the second and third cases together. If this 
code needs clarifying, one way to do it would be to factor out the second check 
into an appropriately-named function.


http://reviews.llvm.org/D8530

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to