[PATCH] D88084: [clang-format] Changed default styles BraceWrappping bool table to directly use variables

[Marcus Johnson via Phabricator via cfe-commits]

MarcusJohnson91 created this revision.
MarcusJohnson91 added reviewers: MyDeveloperDay, sylvestre.ledru.
MarcusJohnson91 added a project: clang-format.
Herald added a project: clang.
MarcusJohnson91 requested review of this revision.

Which should make these defaults more immune to changes in the BraceWrapping enum.

using a table of values is just asking for trouble, and by doing it this way there's more confidence about the correctness of the default styles.

as @Silvestre.Ledru inadvertently pointed out.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88084

Files:
  clang/lib/Format/Format.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88084.293401.patch
Type: text/x-patch
Size: 6586 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200922/3a02ed61/attachment-0001.bin>