> On 28 Mar 2021, at 13:46, Ulrich Mueller <[email protected]> wrote:
> 
> 
>>>>>>> On Sun, 28 Mar 2021, David Seifert wrote:
>> This is just bringing it in line with the rest of the eclass. You know,
>> consistency.
> 
> If that's the goal then the patch should update all occurences, though.

Well, that was definitely the aim. To avoid any doubt, I don’t feel 
particularly passionate about which style we use so it was not the main point 
of this patch. Just copying existing style for consistency.

> Especially those where usage is inconsistent within the same line.

I’m mobile right now but maybe you could show me what you’re referring to? 
Thank you!

We have review for a reason - to improve - but being vague doesn’t help me do 
that ;)

Reply via email to