<div dir="ltr">Looks great, ship it!<div><br></div><div>Wait... one more thing:</div><div><br></div><div>  int [3] *x;</div><div><br></div><div>We should suggest adding parentheses here, because</div><div><br></div><div>  int *x[3];</div>
<div><br></div><div>means the wrong thing.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 22, 2014 at 7:00 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Moved the misplaced bracket logic from ParseDirectDeclarator into its own method.<br>
Use a temporary Declarator to store the bracket location instead of using a custom storage object.<br>
Renamed getDiagLoc to getMissingDeclaratorIdLoc.<br>
Fixed up paren/bracket mistakes.<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D2712" target="_blank">http://reviews.llvm.org/D2712</a><br>
<br>
Files:<br>
  include/clang/Basic/DiagnosticParseKinds.td<br>
  include/clang/Parse/Parser.h<br>
  lib/Parse/ParseDecl.cpp<br>
  test/Parser/brackets.c<br>
  test/Parser/brackets.cpp<br>
</div></div></blockquote></div><br></div>