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
___
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
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
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