From: Kevin Pouget <kevin.pouget@gmail.com>
To: Paul_Koning@dell.com
Cc: gdb-patches@sourceware.org
Subject: Re: Doc: make python function/method descriptions look as in Python
Date: Fri, 21 Oct 2011 14:17:00 -0000 [thread overview]
Message-ID: <CAPftXUKqQMZrLT7-koDsSOpk44pR_uK3cZd5M7MJYQ50gu5W1A@mail.gmail.com> (raw)
In-Reply-To: <09787EF419216C41A903FD14EE5506DD0307F9CCA9@AUSX7MCPC103.AMER.DELL.COM>
On Thu, Sep 15, 2011 at 6:12 PM, <Paul_Koning@dell.com> wrote:
> I committed this change. New diff attached because it changed slightly due to new methods being defined since the last edition.
>
> paul
>
> ChangeLog:
> 2011-09-15 Paul Koning <paul_koning@dell.com>
>
> * gdb.texinfo: Make style of Python functions and methods match
> the syntax of Python. Also put class and module names explicitly
> on function, member, and variable names, matching Python
> documentation conventions.
Hello,
I noticed a little problem with this patch (or maybe it was there
earlier, I didn't check),
with Breakpoint.__init__, the documentation looks like
> @defun Breakpoint.__init__ (spec @r{[}, type @r{[}, wp_class @r{[},internal@r{]]]})
which translate to
> Breakpoint.__init__ (spec [, type [, wp class [,internal]]])
but `internal' doesn't require `wp_class', which doesn't require
`type', do they?
I think it should rather read:
> Breakpoint.__init__ (spec [, type] [, wp class] [,internal])
Cordially,
Kevin
next prev parent reply other threads:[~2011-10-21 13:32 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-01 21:23 Paul_Koning
2011-09-02 12:26 ` Pedro Alves
2011-09-02 16:45 ` Paul_Koning
2011-09-02 17:08 ` Pedro Alves
2011-09-03 19:46 ` Eli Zaretskii
2011-09-03 21:53 ` Matt Rice
2011-09-04 8:38 ` Paul_Koning
2011-09-04 10:37 ` Eli Zaretskii
2011-09-06 15:42 ` Paul_Koning
[not found] ` <09787EF419216C41A903FD14EE5506DD01534916B2@AUSX7MCPC103.AMER.DELL.COM>
2011-09-06 18:01 ` Paul Koning
2011-09-09 21:15 ` Paul Koning
2011-09-09 21:33 ` Eli Zaretskii
2011-09-15 17:19 ` Paul_Koning
2011-10-21 14:17 ` Kevin Pouget [this message]
2011-10-21 15:51 ` Tom Tromey
2011-09-06 19:39 ` Phil Muldoon
2011-09-06 19:45 ` Paul_Koning
2011-09-06 19:57 ` Paul_Koning
2011-09-06 20:07 ` Eli Zaretskii
2011-09-06 20:25 ` Paul_Koning
2011-09-07 0:45 ` Eli Zaretskii
2011-09-06 19:56 ` Eli Zaretskii
2011-09-06 20:01 ` Paul_Koning
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=CAPftXUKqQMZrLT7-koDsSOpk44pR_uK3cZd5M7MJYQ50gu5W1A@mail.gmail.com \
--to=kevin.pouget@gmail.com \
--cc=Paul_Koning@dell.com \
--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