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: [doc RFA] Move docs on auto-loading extensions to Extending GDB section
Date: Mon, 23 Dec 2013 18:45:00 -0000	[thread overview]
Message-ID: <83sitj76h6.fsf@gnu.org> (raw)
In-Reply-To: <m3a9frtpfb.fsf@sspiff.org>

> From: Doug Evans <xdje42@gmail.com>
> Cc: gdb-patches@sourceware.org
> Date: Mon, 23 Dec 2013 10:03:52 -0800
> 
> I did change/add *some* wording, but kept it minimal.
> E.g., I added this:
> 
> +There are various kinds of files @value{GDBN} can automatically load.
> +In addition to these files, @value{GDBN} supports auto-loading code written
> +in various extension languages. @xref{Auto-loading extensions}.
                                 ^^
Two spaces.

> >> +@menu
> >> +* objfile-gdb.ext file::         The @file{@var{objfile}-gdb.@var{ext}} file
> >
> > Node names cannot contain periods, this confuses some Info readers.
> 
> I guess the solution here is to use this?
> 
>      * MENU-ENTRY-NAME: NODE-NAME.   DESCRIPTION
> 
> [cut-n-pasted from texinfo docs]

Yes.

> If so, I'll use it for this entry too:
> 
> * dotdebug_gdb_scripts section:: The @code{.debug_gdb_scripts} section

Thanks.

> >> +For systems using file formats like ELF and COFF,
> >> +when @value{GDBN} loads a new object file
> >> +it will look for a special section named @code{.debug_gdb_scripts}.
> >> +If this section exists, its contents is a list of NUL-terminated names
> >                                                      ^^^
> > "NULL", right?
> 
> NULL is a zero-valued pointer.
> NUL is the common ASCII name for character '\0'.
> I grepped for other uses of NUL in the docs and found a few
> so figured it was ok to use.  E.g.,
> 
> ---snip---
> There is no trailing NUL byte in a remote protocol packet; if the stub
> stores packets in a NUL-terminated format, it should allow an extra
> byte in its buffer for the NUL.  If this stub feature is not supported,
> ---snip---

Unicode calls it NULL, but I won't argue.

Thank, you can commit the patch.

      reply	other threads:[~2013-12-23 18:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-23 17:12 Doug Evans
2013-12-23 17:28 ` Eli Zaretskii
2013-12-23 18:04   ` Doug Evans
2013-12-23 18:45     ` Eli Zaretskii [this message]

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=83sitj76h6.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).