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: binutils@sourceware.org, gdb-patches@sourceware.org
Subject: Re: [PATCH] Add support for embedding scripts in .debug_gdb_scripts.
Date: Fri, 16 Jan 2015 18:02:00 -0000	[thread overview]
Message-ID: <83egqu1u69.fsf@gnu.org> (raw)
In-Reply-To: <CAP9bCMSC0TgsuZ+K0qb6Fkdafh_vbbCL+gBZ3V1h6aM6kUqW+A@mail.gmail.com>

> Date: Fri, 16 Jan 2015 09:15:35 -0800
> From: Doug Evans <xdje42@gmail.com>
> Cc: binutils@sourceware.org, 	"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
> 
> >> +* Python/Guile scripting
> >> +
> >> +** GDB now supports auto-loading of Python/Guile scripts contained in the
> >> +   special section named `.debug_gdb_scripts'.
> >
> > I think we should tell here on what systems/targets this is supported.
> > Otherwise, the NEWS entry is OK.
> 
> The previous news entry didn't, and I'm following that.

So we have no hope for ever fixing past mistakes?

> > Let's be consistent here" "null", lower-case and with 2 'l'.  This is
> > just a plain English word, not an acronym or a C symbol.
> 
> NULL is a typo, thanks for catching that.
> I'm just following usage elsewhere in gdb.texinfo,
> and I like this choice.

So we have no hope for ever fixing past mistakes?

> >> +The rest of the entry, up to the trailing NUL byte,
> >                                              ^^^
> > "null"
> 
> Copied from gdb.texinfo.
> Plus we've had this conversation before. :-)
> 
> https://sourceware.org/ml/gdb-patches/2013-12/msg00892.html
> https://sourceware.org/ml/gdb-patches/2013-12/msg00894.html

So we have no hope for ever fixing past mistakes?

  reply	other threads:[~2015-01-16 18:02 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 [this message]
2015-01-17  1:16       ` Doug Evans
2015-01-17  8:16         ` Eli Zaretskii
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=83egqu1u69.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=binutils@sourceware.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).