[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros
Timothée Fivaz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 11 23:20:24 PDT 2018
TimF added a comment.
In https://reviews.llvm.org/D28462#1064789, @enyquist wrote:
> As far as I know, there are no updates required from me for this pull request-- I rebased on the main trunk recently, and will do it again tonight to be sure. So it should be compiling/working just fine.
> I believe it is just awaiting final approval from somebody.
I have done a checkout of both trunks yesterday, applied your patch and compiled everything. It is working flawlessly and now the defines are correctly aligned. I still hope they'll merge it because I prefer using an official release, but until then it will do the job.
Thanks a lot for your work on this feature, it was the last point preventing us from using clang-format. (embedded development too, so we have lots of defines). And thanks to all the LLVM contributors by the way :-)
Repository:
rL LLVM
https://reviews.llvm.org/D28462
More information about the cfe-commits
mailing list