[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros

Micah S. via Phabricator via llvm-commits llvm-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 llvm-commits mailing list