curdeius added inline comments.
================ Comment at: clang/include/clang/Format/Format.h:533 + /// If ``true``, always break before concept declarations + bool AlwaysBreakBeforeConceptDeclarations; ---------------- It would be nice to have an example here in the doc. ================ Comment at: clang/unittests/Format/FormatTest.cpp:13540 CHECK_PARSE_BOOL(AlignConsecutiveMacros); + CHECK_PARSE_BOOL(AlwaysBreakBeforeConceptDeclarations); CHECK_PARSE_BOOL(AllowAllArgumentsOnNextLine); ---------------- Format: leading whitespace. Should it be in the same "group" as `AlignConsecutiveMacros`? ================ Comment at: clang/unittests/Format/FormatTest.cpp:16745 + " ::std::is_copy_constructable<T> and " + "::std::is_move_constructable<T> and\n" + " requires (T c) {\n" ---------------- Isn't it a strange indentation? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79773/new/ https://reviews.llvm.org/D79773 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits