[PATCH] D55170: [clang-format]: Add NonEmptyParentheses spacing option

Reuben Thomas via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 15 15:18:39 PDT 2019


reuk added a comment.

@MyDeveloperDay I'm not sure you built this branch. Perhaps you applied this patch to an older version of the repo. For me, SpaceBeforeCpp11BracedListOptions is defined at Format.h:1570.

This builds fine for me, on macOS 10.14 with Xcode 10.1's clang. I just rebased onto the most recent clang master and updated llvm. I've been testing/building with this command:

  cmake --build build --target FormatTests && ./build/tools/clang/unittests/Format/FormatTests && cmake --build build --target clang-format

@klimek That's disappointing, I thought other JUCE users might benefit from this patch (it's quite an established library), and the amount of code that I'm a adding is not that high.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55170/new/

https://reviews.llvm.org/D55170





More information about the cfe-commits mailing list