[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros
Micah S. via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 22 10:18:08 PST 2019
micah-s added a comment.
ClamAV recently started using clang-format. We published this blog post about how we're using it: https://blog.clamav.net/2019/02/clamav-adopts-clang-format.html One of the things I called out in the blog post is that we really want this feature, and presently we have to use the "// clang-format on/off" markers throughout our code mostly to keep our consecutive macros aligned.
I haven't found time to build clang-format with this patch to test it on our code base with the markers disabled. If a review of my experience doing so will help get traction for this review in any way, let me know and I'll make the effort.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D28462/new/
https://reviews.llvm.org/D28462
More information about the cfe-commits
mailing list