r212591 - clang-format: Revamp function declaration/definition indentation.
Tobias Grosser
tobias at grosser.es
Wed Jul 9 03:15:58 PDT 2014
On 09/07/2014 11:33, Daniel Jasper wrote:
> On Wed, Jul 9, 2014 at 11:08 AM, Tobias Grosser <tobias at grosser.es> wrote:
>
> a) That is not a lot (<1% of the codebase).
It affects the formatting of almost all cl::opt uses.
>> Sure, would it be OK to revert this change meanwhile to get the build bot
>> clean again.
>>
>
> No. I think it is highly likely, we are going to stick with this change.
Daniel, I believe it is common habit that if someone requests a revert,
this revert is performed and the discussion takes place.
> Moreover, I think the issue here is that we have a buildbot enforcing a
> format based on clang-format's HEAD version.
I believe it is good to understand how clang-format changes over time.
Most changes are insignificant for the existing code so I adjust Polly
immediately. The same holds for style changes that have been agreed on
in the community.
Changes,
Tobias
More information about the cfe-commits
mailing list