Use shorter error messages for whitespace problems
authorWincent Colaiuta <win@wincent.com>
Fri, 14 Dec 2007 11:23:43 +0000 (12:23 +0100)
committerJunio C Hamano <gitster@pobox.com>
Sat, 15 Dec 2007 04:51:58 +0000 (20:51 -0800)
The initial version of the whitespace_error_string() function took the
messages from builtin-apply.c rather than the shorter messages from
diff.c.

This commit addresses Junio's concern that these messages might be too
long (now that we can emit multiple warnings per line).

Signed-off-by: Wincent Colaiuta <win@wincent.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4015-diff-whitespace.sh
ws.c

index 05ef78b2c0875d775461227f8368fac370b20dbc..9bff8f5e4bae6beb22ba31e8be03a0528e06519e 100755 (executable)
@@ -121,7 +121,7 @@ test_expect_success 'check mixed spaces and tabs in indent' '
 
        # This is indented with SP HT SP.
        echo "   foo();" > x &&
-       git diff --check | grep "Space in indent is followed by a tab"
+       git diff --check | grep "space before tab in indent"
 
 '
 
diff --git a/ws.c b/ws.c
index d7d1522f8a177639d3269e5e9a83be39a14a7d89..46cbdd63793aad534d8d8b640df7fd2e86c7d915 100644 (file)
--- a/ws.c
+++ b/ws.c
@@ -101,16 +101,16 @@ char *whitespace_error_string(unsigned ws)
        struct strbuf err;
        strbuf_init(&err, 0);
        if (ws & WS_TRAILING_SPACE)
-               strbuf_addstr(&err, "Adds trailing whitespace");
+               strbuf_addstr(&err, "trailing whitespace");
        if (ws & WS_SPACE_BEFORE_TAB) {
                if (err.len)
                        strbuf_addstr(&err, ", ");
-               strbuf_addstr(&err, "Space in indent is followed by a tab");
+               strbuf_addstr(&err, "space before tab in indent");
        }
        if (ws & WS_INDENT_WITH_NON_TAB) {
                if (err.len)
                        strbuf_addstr(&err, ", ");
-               strbuf_addstr(&err, "Indent more than 8 places with spaces");
+               strbuf_addstr(&err, "indent with spaces");
        }
        return strbuf_detach(&err, NULL);
 }