From: Eli Zaretskii <eliz@gnu.org>
To: Paul_Koning@Dell.com
Cc: gdb-patches@sourceware.org
Subject: Re: Doc: make python function/method descriptions look as in Python
Date: Sat, 03 Sep 2011 19:46:00 -0000 [thread overview]
Message-ID: <837h5p2a6g.fsf@gnu.org> (raw)
In-Reply-To: <09787EF419216C41A903FD14EE5506DD015343BFAE@AUSX7MCPC103.AMER.DELL.COM>
> From: <Paul_Koning@Dell.com>
> Date: Thu, 1 Sep 2011 15:52:59 -0500
>
> I found the descriptions of the Python methods/functions hard to read because they are not shown in Python syntax. The attached patch fixes that. Parentheses now appear wherever they would in the Python code. Also, multiple optional arguments are shown as [arg1 [, arg2]] since by Python rule you can omit only consecutive arguments from the right end. (Having them individually bracketed makes it appear that they can be individually omitted.)
I have no opinion on whether this is desirable or not; I will go with
whatever others think. Python is a read-only language for me.
On the pure Texinfo side, in addition to the typo spotted by Pedro, I
have the following comments:
> -@defun execute command [from_tty] [to_string]
> +@defun execute (command @r{[}, from_tty @r{[}, to_string@r{]}@r{]})
You can mark several `]' characters together, like this:
@defun execute (command @r{[}, from_tty @r{[}, to_string@r{]]})
There are several of these in the patch. What you did is not wrong,
though.
> -@defmethod Value string @r{[}encoding@r{]} @r{[}errors@r{]} @r{[}length@r{]}
> +@defmethod Value string (@r{[}encoding@r{[}, errors@r{[},
> +length@r{]}@r{]}@r{]})
You cannot break any of the @def* lines, they all must be a single
line, even if it is long (here and elsewhere).
Finally, please note that the way you put the parentheses (..) in the
@def* directives, they will be typeset in slant typeface (I think).
So please make a PDF or PS output and eyeball it to make sure you like
the result. If you don't like it, you can use @r{(} and @r{)} like we
do with square brackets.
Thanks.
next prev parent reply other threads:[~2011-09-03 10:27 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 [this message]
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
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=837h5p2a6g.fsf@gnu.org \
--to=eliz@gnu.org \
--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