> See the attached images. 'indent_4' is the current formatting, and
> 'indent_2' is formatting done using 'yapf' with chromium style (2
> spaces) and column limit 78.
Thanks. I still prefer 2-space indentation; for deeper nesting levels
it is easier to stay within the 78 (or 79) character-per-line limit...
> The auto formatters (both yapf and black), however, perform somewhat
> undesirable changes at places. See 'regex_indent_old' and
> 'regex_indent_new' as an example.
Ouch. Looks like bugs in the formatters...
>> In PEP 8, value 4 for indentation is like the parallel postulate:
>> it is completely isolated from all other recommendations...
>
> I'm not sure what you mean by 'isolated' here, [...]
I mean: If you change this value, it doesn't have any consequence to
the remaining rules (the same is true for Euclid's parallel
postulate).
> I'll wait before applying a formatter till there is a consensus
> here :-)
Indeed. You shouldn't waste time on this issue since you already
maintain well-formatted code.
Werner
_______________________________________________
Freetype-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype-devel
