Page MenuHomeFreeBSD

style: tweak tab after #define advice
ClosedPublic

Authored by imp on May 13 2021, 9:53 PM.

Details

Summary

Once upon a time, #define<tab> was cultural thing. However, even when it
was promulgated, it was a minority usage. 20 years ago the split was
30k/69k (tab/space) and today the split is 80k/546k (tab/space). Update
guidance to allow either.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

imp requested review of this revision.May 13 2021, 9:53 PM
This revision is now accepted and ready to land.May 13 2021, 10:01 PM

Can we please encourage per-file or per-group consistency? Patches are annoying to read when the a group of #defines randomly switches between styles.

Can we please encourage per-file or per-group consistency? Patches are annoying to read when the a group of #defines randomly switches between styles.

Ditto @brooks ; Perhaps a blanket statement near the top that when making minor modifications to existing files, go with the flow, even if it's not strictly conformant to style(9)?

add suggested consistency stuff

This revision now requires review to proceed.May 27 2021, 2:16 PM
jhb added inline comments.
share/man/man9/style.9
220

I would probably drop the "try to".

This revision is now accepted and ready to land.Jun 11 2021, 4:37 PM
This revision was automatically updated to reflect the committed changes.