Hi all,

James and I would like to make this small, but powerful, change to clang-format:

        -AlignConsecutiveAssignments: false
        +AlignConsecutiveAssignments: true


This will try to align assignments such that the = signs line up. E.g.

enum {
  XHEADER_X_CACHE_KEY  = 0x0004u,
  XHEADER_X_MILESTONES = 0x0008u,
  XHEADER_X_CACHE      = 0x0010u,
  XHEADER_X_GENERATION = 0x0020u,
  XHEADER_X_TXN_UUID   = 0x0040u,
};

This is a pretty significant change (about 12k lines in 650 files), but I think 
it’s worth it. And again, since we have clang-format, getting your old branches 
up to this new indentation is easy, just temporarily copy over the new 
.clang-format, and re-run “make clang-format", and voila, you are up-to-date 
with master as far as formatting goes.

Please voice concerns with this now. To make the LTS release manageable and 
easy, we’d also back port this to the next RC1 for v6.2.0. I have spoken with 
He-Who-Shall-Make-All-LTS-Releases, and he’s +1 on this idea as well.

Cheers,

— leif

Reply via email to