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
