Serg Tereshchenko <serg.partizan@xxxxxxxxx> writes:
> Hello.
>
> I want to improve styling of git-citool, so it supports dark themes.
> But first i want to remove "mixed indent warning" from my editor
> modeline.
>
> This patch does not change anything besides indents, and sometimes i
> replace
>
> ```tcl
> if {long ||
> multiline ||
> statement}
> ```
>
> with more readable
>
> ```tcl
> if {
> long
> || multiline
> || statement
> }
> ```
> But only in lines with mixed indents.
You don't ever want to do that in a patch for "fix indentation"
patch.
Besides, the comparison between these two styles is subjective, and
you do not represent the majority view of who needs to work on the
code in git-gui, so "with more readable" is not a very good
justification to do so.
If the existing code in the whole file uses mixture of both styles,
and one style is used overwhelmingly more than the other, a
follow-up patch to adjust the style to one, with "consistency" as
the justification, would be a good idea, though.
Thanks.
