public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Doug Evans <xdje42@gmail.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH] Add support for embedding scripts in .debug_gdb_scripts.
Date: Sat, 17 Jan 2015 08:16:00 -0000	[thread overview]
Message-ID: <8361c5254p.fsf@gnu.org> (raw)
In-Reply-To: <CAP9bCMREvQTdNiH_fP8r3UUynFevR9DNw2nc8ESJ7r0qnF9boQ@mail.gmail.com>

> Date: Fri, 16 Jan 2015 17:15:49 -0800
> From: Doug Evans <xdje42@gmail.com>
> Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
> 
> On Fri, Jan 16, 2015 at 10:01 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> > So we have no hope for ever fixing past mistakes?
> 
> Past mistakes?
> .debug_gdb_scripts isn't that old of a feature.

I wasn't talking only about that.  I was talking in general about the
argument "we do this elsewhere 'like this', so let's continue doing
that 'like this'".  For example, the "NUL" thingie.

> Maybe we need more formal community-agreed-on
> conventions and rules for NEWS and docs.

Maybe we should, but I'd like first to agree that an argument of this
kind doesn't have too much weight.  It's okay to look at past
practices when the choice is purely stylistic.  But when there are
clear advantages to deviating from past practices, those past
practices shouldn't hold us back, otherwise we will stagnate.  Agreed?

In this case, "NUL" is simply incorrect English: there's no such word
or acronym.  The only legitimate use of "NUL" I know of is in
reference to the DOS/Windows null device.

As for showing the systems where .debug_gdb_scripts feature is
supported, there are clear advantages to providing that information in
NEWS, and the price is quite low, I hope you will agree.

I can also live with you asking me in response to please change all
the other instances to use the same style.  But what I would prefer
not to live with is flat refusal to make a requested change in your
patch because "we do that elsewhere".

> These things seem to be of a "shall be this way" flavor,
> and I wasn't expecting that.

Isn't that normal during patch review process?

> When I cut-n-paste from code I can usually
> tell what's expected, but I can't do that for NEWS/doc,

From experience, my requests are remarkably consistent, even if the
same issues pop up with quite some time in-between.  You've just cited
a similar discussion about NUL from more than a year ago, which I
didn't event remember.

> [Imagine if coding conventions changed like this.]

They do, because standards.texi is actively maintained.  Things I've
read and memorized years ago have changed, sometimes radically so, and
I keep bumping into them when people say my code isn't according to
GCS and cite from there.  That's life, and I accept it.

  reply	other threads:[~2015-01-17  8:16 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-15 17:32 Doug Evans
2015-01-15 18:11 ` Eli Zaretskii
2015-01-16 17:15   ` Doug Evans
2015-01-16 18:02     ` Eli Zaretskii
2015-01-17  1:16       ` Doug Evans
2015-01-17  8:16         ` Eli Zaretskii [this message]
2015-01-18  4:16           ` Doug Evans
2015-01-18 16:23             ` Eli Zaretskii
2015-01-18 20:48               ` Doug Evans
2015-01-19 14:49                 ` Joel Brobecker
2015-01-20 16:35                   ` Doug Evans
2015-01-21  9:57                     ` Joel Brobecker
2015-02-13 16:15                     ` Stan Shebs
2015-02-13 16:45                       ` Eli Zaretskii
2015-02-13 16:46                       ` Andreas Schwab
2015-02-13 18:05                     ` Pedro Alves
2015-02-15 11:53                       ` Corinna Vinschen
2015-01-19 16:05                 ` Eli Zaretskii
2015-01-31 23:31 ` Doug Evans

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=8361c5254p.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=xdje42@gmail.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 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).