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

[Krasimir Georgiev via Phabricator via cfe-commits]

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



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