[PATCH] D37845: [clang-format] New flag - BraceWrapping.AfterExternC

2017-09-14 Thread Pawel Maciocha via Phabricator via cfe-commits
PriMee updated this revision to Diff 115192. PriMee added a comment. Done :) https://reviews.llvm.org/D37845 Files: docs/ClangFormatStyleOptions.rst include/clang/Format/Format.h lib/Format/Format.cpp lib/Format/UnwrappedLineFormatter.cpp lib/Format/UnwrappedLineParser.cpp unittests

[PATCH] D37845: [clang-format] New flag - BraceWrapping.AfterExternC

2017-09-14 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir added inline comments. Comment at: docs/ClangFormatStyleOptions.rst:664 + * ``bool AfterExternC`` Wrap extern "C" blocks. + I think this is overly specific. The C++ standard also [[ http://en.cppreference.com/w/cpp/language/language_linkage | allows

[PATCH] D37845: [clang-format] New flag - BraceWrapping.AfterExternC

2017-09-14 Thread Pawel Maciocha via Phabricator via cfe-commits
PriMee created this revision. Herald added a subscriber: klimek. Bug: https://bugs.llvm.org/show_bug.cgi?id=34016 - **"extern C part"** **Problem:** Due to the lack of "brace wrapping extern" flag, clang format does parse the block after **extern** keyword moving the opening bracket to the head