[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-05 Thread Björn Schäpers via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc2257fe23672: [clang-format] Fix SeparateDefinitionBlocks docs and ... (authored by HazardyKnusperkeks). Repository: rG LLVM Github Monorepo CHAN

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius accepted this revision. curdeius added a comment. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116557/new/ https://reviews.llvm.org/D116557 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread ksyx via Phabricator via cfe-commits
ksyx accepted this revision. ksyx added a comment. This revision is now accepted and ready to land. Thanks for the fix! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116557/new/ https://reviews.llvm.org/D116557 ___ cfe-commits mailing list cf

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. In D116557#3218328 , @ksyx wrote: > Thanks for the fix :) No Problem. Comment at: clang/docs/ReleaseNotes.rst:300 - Option ``SeparateDefinitionBlocks`` has been added to insert or remove empty

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks updated this revision to Diff 397148. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116557/new/ https://reviews.llvm.org/D116557 Files: clang/docs/ClangFormatStyleOptions.rst clang/include/clang/Format/Format.h Index: clang/include/clang/Format/Format.h =

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread ksyx via Phabricator via cfe-commits
ksyx added a comment. Thanks for the fix :) Comment at: clang/docs/ReleaseNotes.rst:300 - Option ``SeparateDefinitionBlocks`` has been added to insert or remove empty lines between definition blocks including functions, classes, structs, enums, The change

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. Without the enum documentation I got $ ./clang/docs/tools/dump_format_style.py Traceback (most recent call last): File "/c/GIT/llvm-dev/./clang/docs/tools/dump_format_style.py", line 344, in opts = OptionsReader(f).read_options() File "/c/GI

[PATCH] D116557: [clang-format] Fix SeparateDefinitionBlocks docs and ...

2022-01-03 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks created this revision. HazardyKnusperkeks added reviewers: ksyx, MyDeveloperDay, curdeius. HazardyKnusperkeks added a project: clang-format. HazardyKnusperkeks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. the Style's e