[PATCH] Improve diagnostic message for misplaced square brackets

[Richard Smith]

Looks great, with one tweak.

================
Comment at: lib/Parse/ParseDecl.cpp:5646-5649
@@ +5645,6 @@
+
+  // Nothing after the brackets has been parsed as part of the Declarator, so
+  // no correction is needed.
+  if (OldEndLoc == D.getLocEnd())
+    return;
+
----------------
This looks unreachable. If nothing after the brackets has been parsed, then there can't be an identifier, so we should have bailed out already. Remove?

http://reviews.llvm.org/D2712