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