ank added inline comments.

================
Comment at: unittests/Format/FormatTest.cpp:4449-4450
+               "  })\n"
+               "      
.foo(\"aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa\"\n"
+               "           \"bbbbbbbbbbbbbbbbbbbbbb\");\n"
+               "}");
----------------
klimek wrote:
> One more nit: I'd use a Style with a smaller column limit for these tests, 
> makes them a lot more readable :)
makes sense, I'll do that.


================
Comment at: unittests/Format/FormatTest.cpp:4452
+               "}");
+
+  verifyFormat("test() {\n"
----------------
klimek wrote:
> Are the ones below here testing anything that's not tested above?
nope I'll remove them!


Repository:
  rC Clang

https://reviews.llvm.org/D45719



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

Reply via email to