[PATCH] D75410: [clang-format] Fixed BraceWrapping.AfterExternBlock

2020-03-03 Thread Marcus Johnson via Phabricator via cfe-commits
MarcusJohnson91 abandoned this revision. MarcusJohnson91 added a comment. I'm moving the intended change to a new clang-format option instead of modifying an established one. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75410/new/ https://reviews.llvm.org/D75410 ___

[PATCH] D75410: [clang-format] Fixed BraceWrapping.AfterExternBlock

2020-03-01 Thread Marcus Johnson via Phabricator via cfe-commits
MarcusJohnson91 updated this revision to Diff 247524. MarcusJohnson91 added a comment. Full Context Diff (I think?) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75410/new/ https://reviews.llvm.org/D75410 Files: clang/docs/ClangFormatStyleOptions.rst clang/include/clang/Format/Form

[PATCH] D75410: [clang-format] Fixed BraceWrapping.AfterExternBlock

2020-03-01 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. This needs a full context diff, before we can even look at it properly. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D7

[PATCH] D75410: [Clang-Format] Fixed BraceWrapping.AfterExternBlock

2020-02-29 Thread Marcus Johnson via Phabricator via cfe-commits
MarcusJohnson91 updated this revision to Diff 247462. MarcusJohnson91 added a comment. Rewrote the AfterExternBlock code to rely on just that clang-format option and remove the ExternNamespace check. Fixed the tests as well. and clang-formatted everything CHANGES SINCE LAST ACTION https://r