Re: [PATCH v2 5/8] clang-format: avoid braces on simple single-statement bodies
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- Jul 12, 2024, 15:29 UTC
- Message-ID
- <42867b72-58e0-48d7-86eb-7c312db942dd@gmail.com>
- In-Reply-To
- <xmqq7cdqpry4.fsf@gitster.g>
On 12/07/2024 16:09, Junio C Hamano wrote:
Show 5 quoted lines
> Of course if we really wanted to avoid end-user confusion and still > want to have this in CI (if only to see how well the rule fares, and > what the actual false-positive rate is), we _could_ run CI's job > with custom .clang-format file that is not visible to end-users in > their normal checkout, or something silly like that.
Getting some idea of how useful the auto-formatter is would be valuable I think.
Show 5 quoted lines
> If we are > going to use it, then we should use it everywhere, making sure > everybody is careful. If the cost of forcing everybody to be > careful is too high, we may want to retract it, but we won't know > until we try.
It would be great if we could find a way to auto-format the code without inconveniencing contributors.
Best Wishes
Phillip