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

Reuben Thomas via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 8 14:48:36 PST 2019


reuk updated this revision to Diff 189942.
reuk added a comment.

I've rebased onto master, and removed unrelated formatting changes. I've also tried to remove some of the duplicate parens-related expressions.

I agree that the heavy nested boolean expressions are a bit painful to read, but I'm not sure whether a more general tidy-up falls within the scope of this PR.


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

https://reviews.llvm.org/D55170

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55170.189942.patch
Type: text/x-patch
Size: 8950 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190308/6b07f6f2/attachment-0001.bin>


More information about the cfe-commits mailing list