djasper added inline comments.

================
Comment at: include/clang/Format/Format.h:644
+    /// This option is used only if the opening brace of the function has
+    /// already
+    /// been wrapped, i.e. the `AfterFunction` brace wrapping mode is set, and
----------------
Reflow the comment.



================
Comment at: include/clang/Format/Format.h:654
+    ///
+    bool AllowEmptyFunctionBody;
     /// \brief Wrap before ``catch``.
----------------
I think the name probably isn't very intuitive. Maybe invert it and call it 
"SplitEmptyFunctionBody"?


================
Comment at: unittests/Format/FormatTest.cpp:6092
+  verifyFormat("int f()\n"
+                          "{\n"
+               "  return 42;\n"
----------------
indent. Here and elsewhere.


https://reviews.llvm.org/D33447



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

Reply via email to