]> git.ipfire.org Git - thirdparty/git.git/commitdiff
.gitattributes: remove misspelled no-op whitespace attribute
authorJunio C Hamano <gitster@pobox.com>
Tue, 11 Nov 2025 18:41:20 +0000 (10:41 -0800)
committerJunio C Hamano <gitster@pobox.com>
Tue, 11 Nov 2025 18:53:37 +0000 (10:53 -0800)
Ever since 14f9e128 (Define the project whitespace policy,
2008-02-10) added the whitespace rules to .gitattributes, we spelled
the most general rule like so:

    * whitespace=!indent,trail,space

in the top-level .gitattributes file.  The intent of this line was
described in the commit log message:

     - Unless otherwise specified, indent with SP that could be
       replaced with HT are not "bad".  But SP before HT in the
       indent is "bad", and trailing whitespaces are "bad".

It clearly wanted to disable indent-with-non-tab, so !indent is most
likely a misspelt form of '-indent'.  Because indent-with-non-tab
has never been enabled by default, by luck this was not causing any
ill effect.

We could either remove "!indent", or spell it "-indent".  The
immediate effect would be the same.  It would only start to make a
difference when/if we enable indent-with-non-tab by default in
future versions of Git.

Let's take the former option to remove "!indent" from the list.  We
would feel the effect first-hand ourselves before anybody else if we
ever decide to change the built-in default whitespace rules, which
would be hidden from us if we decide to rewrite it to "-indent"
instead.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
.gitattributes

index 158c3d45c4c10cb19c7d3fb02d6bfdaf7a1794f1..2a50ebaf2ee149cd63de2851e9137bf71008ef16 100644 (file)
@@ -1,4 +1,4 @@
-* whitespace=!indent,trail,space
+* whitespace=trail,space
 *.[ch] whitespace=indent,trail,space diff=cpp
 *.sh whitespace=indent,trail,space text eol=lf
 *.perl text eol=lf diff=perl