Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: gdb-patches@sourceware.org
Subject: Re: [rfc] Implementation of qXfer
Date: Thu, 22 Jun 2006 20:07:00 -0000	[thread overview]
Message-ID: <u7j39ar8p.fsf@gnu.org> (raw)
In-Reply-To: <20060622033247.GA27704@nevyn.them.org> (message from Daniel 	Jacobowitz on Wed, 21 Jun 2006 23:32:47 -0400)

> Date: Wed, 21 Jun 2006 23:32:47 -0400
> From: Daniel Jacobowitz <drow@false.org>
> 
> Eli, I had to do something mildly surprising with texinfo xrefs: you can
> reference an anchor which contains colons, but you have to use at least a
> two-argument form of @xref.  If you use the one-argument form, the "info"
> browser will choke trying to parse it.  I wanted to keep the anchor named
> after the packet description it was attached to.  Do the docs look OK?

They are OK, except for this issue with anchors whose names contain a
colon: this is against the Texinfo language rules.  You might be lucky
with a particular Info reader, but there are others out there, and
each one of them uses a different method of searching for
cross-reference names (some use fixed strings, others use various
regular expressions).  Using a colon makes them fail in different
situations and in different interesting ways.

So please let's remove the colons and use some other mnemonic methods
to make the xref reflect the packet descriptor.

> [I realize they may be a bit tricky to review; the qXfer docs are large,
> and they replace the qPart docs but sit somewhere else in the alphabetical
> list of packets.

AFAICS, you just moved the text elsewhere and replaced qPart with
qXfer, right?  If there are new portions of text, please tell where
they are.


  reply	other threads:[~2006-06-22 20:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-22  3:32 Daniel Jacobowitz
2006-06-22 20:07 ` Eli Zaretskii [this message]
2006-06-22 20:13   ` Daniel Jacobowitz
2006-06-23  7:34     ` Eli Zaretskii
2006-07-05 19:15 ` Daniel Jacobowitz
2006-07-12 18:51   ` Daniel Jacobowitz

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=u7j39ar8p.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    /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