Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Thiago Jung Bauermann <bauerman@br.ibm.com>
Cc: Eli Zaretskii <eliz@gnu.org>, gdb-patches@sourceware.org
Subject: Re: [RFC][python] Fixes and improvements to gdb.Value.
Date: Mon, 05 Jan 2009 21:01:00 -0000	[thread overview]
Message-ID: <m38wppjwmy.fsf@fleche.redhat.com> (raw)
In-Reply-To: <1231024147.8380.166.camel@localhost.localdomain> (Thiago Jung Bauermann's message of "Sat\, 03 Jan 2009 21\:09\:07 -0200")

>>>>> "Thiago" == Thiago Jung Bauermann <bauerman@br.ibm.com> writes:

>> > +@findex gdb.history
>> 
>> The text below this does not mention "gdb.history" at all.  Should it?
>> I could imagine a reader who gets here by following the "gdb.history"
>> index entry, and is then puzzled by not finding that text anywhere.

Thiago> Well, the text is in the section which describes functions in the gdb
Thiago> module, so the reader should be aware that the function is inside it.
Thiago> Also, in Python "gdb.history" wouldn't be the only possible way of
Thiago> referencing the function. If the user imports the gdb module with
Thiago> "from gdb import *", then he would just need to type "history" to use
Thiago> it. "gdb.history" is only special in that from the CLI, the gdb module
Thiago> is automatically imported and would be accessible to the user spelled
Thiago> like that. But (s)he could type "python from gdb import *" and then use
Thiago> the other way to access it.

Thiago> In a nutshell, my answer is "no, I don't think it should". :-)

I agree.

Note that the existing Python docs in the manual already use this
approach.  This patch merely continues it.

My reason for adding these index entries is that a Python programmer
might reasonably expect to find the fully-qualified function name in
the manual's index.  However, it is strange to constantly see the
fully-qualified name in the documentation text.  So, I used the short
form in the text and added index entries.

Thiago -- I know it is kind of a pain, but would you mind putting any
changes you make back onto the Python branch?  Or if you can't do it,
let me know and I will try to.  I'd like to keep the branch
"canonical" so that we can diff against mainline to see what Python
bits ought to be merged.

Tom


  parent reply	other threads:[~2009-01-05 21:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-03  2:25 Thiago Jung Bauermann
2009-01-03  9:42 ` Eli Zaretskii
2009-01-03 23:09   ` Thiago Jung Bauermann
2009-01-04  4:20     ` Eli Zaretskii
2009-01-05 21:01     ` Tom Tromey [this message]
2009-01-29 14:16       ` Thiago Jung Bauermann
2009-02-02 13:06 [RFA][python] Fixes for existing Python code Thiago Jung Bauermann
2009-02-02 18:30 ` Tom Tromey
2009-02-04  1:01   ` [RFC][python] Fixes and improvements to gdb.Value. (was Re: [RFA][python] Fixes for existing Python code.) Thiago Jung Bauermann
2009-02-04 19:40     ` [RFC][python] Fixes and improvements to gdb.Value Tom Tromey
2009-02-04 21:57       ` Thiago Jung Bauermann

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=m38wppjwmy.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=bauerman@br.ibm.com \
    --cc=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