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

[Chris Beck via cfe-commits]

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.


http://reviews.llvm.org/D11851

Files:
  include/clang/Format/Format.h
  lib/Format/Format.cpp
  lib/Format/UnwrappedLineFormatter.cpp
  unittests/Format/FormatTest.cpp

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