Thread (5 messages) flat view 5 messages, 3 authors, 2020-09-22

Re: style(git-gui): Fix mixed tabs & spaces; Always use tabs.

From: Junio C Hamano <hidden>
Date: 2020-08-22 19:27:07

Possibly related (same subject, not in this thread)

Serg Tereshchenko [off-list ref] 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
if {long ||
    multiline ||
    statement}
with more readable
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help