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.net \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.