From: "Wesley J. Landaker" <wjl@icecavern•net>
To: Junio C Hamano <gitster@pobox•com>
Cc: git@vger•kernel.org
Subject: Re: [PATCH] Documentation: indent-with-non-tab uses tabwidth setting, not just 8
Date: Mon, 17 Sep 2012 08:20:28 -0600 [thread overview]
Message-ID: <201209170820.28489.wjl@icecavern.net> (raw)
In-Reply-To: <7v8vc9w5hq.fsf@alter.siamese.dyndns.org>
On Monday, September 17, 2012 00:03:29 Junio C Hamano wrote:
> An alternative would be to lose the "8" (or `tabwidth`) from that
> description. I've always thought that the description of `tabwidth`
> is clear enough that "8" in the patch is not a hardcoded non-overridable
> value but is merely a default, but after reading that section a few
> more times, I no longer think that is the case.
>
> I originally wrote "8 or more space" but that wasn't because I
> thought it was important to stress "8 is the default", but because I
> didn't think of a better way to say what I wanted to say, which was
> "if you are filling the indentation with spaces when you could have
> just typed a tab with a few spaces, this error triggers", in other
> words "use of this is to encourage indenting with tabs".
Okay, I'm going to generated a new patch that hopefully reads better!
next prev parent reply other threads:[~2012-09-17 14:20 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-16 17:12 [PATCH] Documentation: indent-with-non-tab uses tabwidth setting, not just 8 Wesley J. Landaker
2012-09-17 5:16 ` Junio C Hamano
2012-09-17 5:41 ` Wesley J. Landaker
2012-09-17 6:03 ` Junio C Hamano
2012-09-17 14:20 ` Wesley J. Landaker [this message]
2012-09-17 14:22 ` [PATCH] Documentation: indent-with-non-tab uses "equivalent tabs" not 8 Wesley J. Landaker
2012-09-17 17:23 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201209170820.28489.wjl@icecavern.net \
--to=wjl@icecavern$(echo .)net \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox