owenpan added inline comments.

================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:3163
 
+    case tok::identifier:
     default:
----------------
cjdb wrote:
> owenpan wrote:
> > Do we still need this line?
> I'd prefer to get rid of `default` if we can.
We can't. It's the whole point of the fix.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120774/new/

https://reviews.llvm.org/D120774

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to