public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Richard Guenther <richard.guenther@gmail.com>
To: Mike Stump <mikestump@comcast.net>
Cc: Gerald Pfeifer <gerald@pfeifer.com>,
	"Joseph S. Myers" <joseph@codesourcery.com>,
		Michael Witten <mfwitten@gmail.com>,
	gcc-patches@gcc.gnu.org
Subject: Re: [PATCH 2/4] Docs: extend.texi: Remove trailing blanks from lines
Date: Thu, 05 May 2011 09:42:00 -0000	[thread overview]
Message-ID: <BANLkTinNvfU-jss7MUKXHKzDDK-KTzSNgA@mail.gmail.com> (raw)
In-Reply-To: <4D15F1D1-FAC0-4978-ADA4-DCD79197566D@comcast.net>

On Thu, May 5, 2011 at 1:57 AM, Mike Stump <mikestump@comcast.net> wrote:
> On May 4, 2011, at 11:40 AM, Gerald Pfeifer wrote:
>> Documentation may be a bit more relaxed, and if you have one doc
>> maintainer approve and the other abstain that may be more boring
>> that you might hope for. :-)
>
> Actually, I was aiming for a global person to ack gcc/*...  This could be less boring that just gcc/doc.  :-)

I think I was the one complaining loudest when HJ checked in his patch.
And I still think such patches are a) pointless b) cause harm to some
subjective degree.  Coding style rules are nice, but if they cause any
degree of harm when enforcing them they are IMHO only guildelines,
not hard rules (after all, if they were we'd have a commit hook that verifies
new violations do not slip in).

And yes, in "harm" I mostly refer to svn blame annoyances and to
patch/branch merge conflicts where context that causes the conflict
only changed in whitespace.

Richard.

  reply	other threads:[~2011-05-05  9:33 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-28  1:22 [PATCH 0/4] Docs: extend.texi Michael Witten
2011-04-28  1:30 ` [PATCH 1/4] Docs: extend.texi: Add missing semicolon for consistency Michael Witten
2015-04-08 19:13   ` Gerald Pfeifer
2015-04-12  1:14     ` Michael Witten
2015-04-12 20:22       ` Gerald Pfeifer
2011-04-28  1:34 ` [PATCH 2/4] Docs: extend.texi: Remove trailing blanks from lines Michael Witten
2011-05-03 20:56   ` Gerald Pfeifer
2011-05-04  6:31     ` Michael Witten
2011-05-04 18:05       ` Mike Stump
2011-05-04 18:39         ` Nathan Froyd
2011-05-04 10:13     ` Joseph S. Myers
2011-05-04 18:34       ` Mike Stump
2011-05-04 18:38         ` Joseph S. Myers
2011-05-04 18:42         ` Gerald Pfeifer
2011-05-04 18:58           ` Joseph S. Myers
2011-05-04 23:57             ` Mike Stump
2011-05-05  0:15           ` Mike Stump
2011-05-05  9:42             ` Richard Guenther [this message]
2011-05-05 18:12               ` Michael Witten
2011-04-28  1:36 ` [PATCH 4/4] Docs: extend.texi: Reword and rearrange attribute node introductions Michael Witten
2011-04-28  1:38 ` [PATCH 3/4] Docs: extend.texi: Rearrange nodes; no text was removed or added Michael Witten
2011-05-21 12:24 ` [PATCH 0/4] Docs: extend.texi Michael Witten
2011-06-10  2:06 ` Michael Witten
2011-06-20 17:20 ` Michael Witten
2011-06-23  8:00 ` Michael Witten
2011-06-27 21:18   ` Michael Witten
2011-06-27 21:36 ` Michael Witten
2011-06-30 22:34 ` Michael Witten

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=BANLkTinNvfU-jss7MUKXHKzDDK-KTzSNgA@mail.gmail.com \
    --to=richard.guenther@gmail.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gerald@pfeifer.com \
    --cc=joseph@codesourcery.com \
    --cc=mfwitten@gmail.com \
    --cc=mikestump@comcast.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).