[PATCH] D40068: Implement more accurate penalty & trade-offs while breaking protruding tokens.

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 17 02:42:36 PST 2017


krasimir added inline comments.


================
Comment at: lib/Format/ContinuationIndenter.cpp:1616
+
+
       if (!DryRun)
----------------
nit: remove an empty line


================
Comment at: unittests/Format/FormatTest.cpp:9951
+                   "\n*/",
+                   Style));
+
----------------
Why didn't this get reformatted as:
```
  EXPECT_EQ("int a; /* first line\n"
            "        * second line\n"
            "        * third line\n"
            "        */",
```


https://reviews.llvm.org/D40068





More information about the cfe-commits mailing list