{"thread":{"id":"27690","subject":"[PATCH] Documentation: git diff --check respects core.whitespace","startedAt":"2011-06-22T15:33:02Z","lastAt":"2011-06-22T23:12:16Z","messageCount":3,"participants":["Christof Krüger","Sverre Rabbelier","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"170472","messageId":"1308756782-30241-1-git-send-email-git@christof-krueger.de","threadId":"27690","inReplyTo":null,"subject":"[PATCH] Documentation: git diff --check respects core.whitespace","fromName":"Christof Krüger","fromEmail":"git@christof-krueger.de","sentAt":"2011-06-22T15:33:02Z","receivedAt":"2011-06-22T15:33:02Z","isPatch":true,"sender":{"key":"git@christof-krueger.de","avatar":null},"body":"Fix documentation on \"git diff --check\" by adopting the description from\n\"git apply --whitespace\".\n\nSigned-off-by: Christof Krüger <git@christof-krueger.de>\n---\n Documentation/diff-options.txt |   12 ++++++++----\n 1 files changed, 8 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\nindex c7ed946..24f189f 100644\n--- a/Documentation/diff-options.txt\n+++ b/Documentation/diff-options.txt\n@@ -224,10 +224,14 @@ endif::git-format-patch[]\n \n ifndef::git-format-patch[]\n --check::\n-\tWarn if changes introduce trailing whitespace\n-\tor an indent that uses a space before a tab. Exits with\n-\tnon-zero status if problems are found. Not compatible with\n-\t--exit-code.\n+\tWarn if changes introduce whitespace errors.  What are\n+\tconsidered whitespace errors is controlled by `core.whitespace`\n+\tconfiguration.  By default, trailing whitespaces (including\n+\tlines that solely consist of whitespaces) and a space character\n+\tthat is immediately followed by a tab character inside the\n+\tinitial indent of the line are considered whitespace errors.\n+\tExits with non-zero status if problems are found. Not compatible\n+\twith --exit-code.\n endif::git-format-patch[]\n \n --full-index::\n-- \n1.7.6.rc2\n"},{"id":"170482","messageId":"BANLkTik-zxSTd-hJjsdjuc7m9XE2PLUsQQ@mail.gmail.com","threadId":"27690","inReplyTo":"1308756782-30241-1-git-send-email-git@christof-krueger.de","subject":"Re: [PATCH] Documentation: git diff --check respects core.whitespace","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2011-06-22T21:43:38Z","receivedAt":"2011-06-22T21:43:38Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\n2011/6/22 Christof Krüger <git@christof-krueger.de>:\n> What are considered whitespace errors is controlled by `core.whitespace`\n\nAm I the only one who fees that \"What are considered <plural>\" would\nsound better if phrased \"What is considered a <singular>\"? Is it\nbecause I haven't heard the phrase much, or is the latter actually\nbetter somehow?\n\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"170483","messageId":"7vwrgd4g1b.fsf@alter.siamese.dyndns.org","threadId":"27690","inReplyTo":"BANLkTik-zxSTd-hJjsdjuc7m9XE2PLUsQQ@mail.gmail.com","subject":"Re: [PATCH] Documentation: git diff --check respects core.whitespace","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-06-22T23:12:16Z","receivedAt":"2011-06-22T23:12:16Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sverre Rabbelier <srabbelier@gmail.com> writes:\n\n> Heya,\n>\n> 2011/6/22 Christof Krüger <git@christof-krueger.de>:\n>> What are considered whitespace errors is controlled by `core.whitespace`\n>\n> Am I the only one who fees that \"What are considered <plural>\" would\n> sound better if phrased \"What is considered a <singular>\"? Is it\n> because I haven't heard the phrase much, or is the latter actually\n> better somehow?\n\nPerhaps. In any case, the original description seems to have been\ninadequate for somebody to understand what it does, so the patch is an\nimprovement; I've queued it.\n\nThanks.\n"}]}