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.
next prev parent 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