sstwcw marked 3 inline comments as done.
sstwcw added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:4525
-    const FormatToken *Next = Right.getNextNonComment();
-    if (!Next || Next->is(tok::semi))
       return false;
----------------
MyDeveloperDay wrote:
> how is the semi case handled or is it not needed
It is not needed anymore.  It was added to handle a semicolon following a goto 
label. Now the type `TT_GotoLabelColon` includes the case.


================
Comment at: clang/unittests/Format/FormatTest.cpp:3020-3024
   verifyFormat("{\n"
                "  some_code();\n"
-               "test_label:;\n"
-               "  int i = 0;\n"
-               "}");
----------------
HazardyKnusperkeks wrote:
> Why did you remove that?
It was an accident.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148484

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

Reply via email to