[PATCH] D84090: [clang-format] Add BitFieldColonSpacing option

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 21 02:08:27 PDT 2020


MyDeveloperDay added a subscriber: djasper.
MyDeveloperDay added a comment.

> Thanks for assistance and review. And sorry about the extra noise.

No worries on my part about the noise, firstly this was a really nice clean, quick review but also the noise is part of the learning for us all (especially for me!), I'd much rather spend the time helping to build a `clang-formatters` community like this with a little noise. In my view it is important for us to build a strong community who can help take over the mantel from @djasper and @klimek's amazing work and let them move onto other areas of interest hopefully with the knowledge we won't undo all their good work!

if its OK with you I'd like to keep you on my list of people who I can use as a reviewer, I very much appreciated your analytical approach to determine what was the most used options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84090





More information about the cfe-commits mailing list