[PATCH] D11851: clang-format: Add "AllowShortNamespacesOnASingleLine" option

Chris Beck via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 16 12:32:06 PDT 2015

cbeck88 removed rL LLVM as the repository for this revision.
cbeck88 updated this revision to Diff 32249.
cbeck88 added a comment.

- Fixed formatting in header
- Fixed handling of nested namespaces
- Added unit tests

djasper: I thought about it, I guess I don't have a particularly good reason that the namespaces logic should be a separate function from "tryMergeSimpleBlock", and maybe it should indeed be refactored to be part of the case analysis of that function. When I began I thought this would just be simpler and make it easier to customize the namespace handling vs. other blocks, but now I'm less sure of this. At least with the way I'm now trying to handle the nested namespaces, this seems a bit simpler, but maybe that part should be changed.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11851.32249.patch
Type: text/x-patch
Size: 8428 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150816/9fd8dd76/attachment.bin>

More information about the cfe-commits mailing list