Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Khoo Yit Phang <khooyp@cs.umd.edu>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Khoo Yit Phang <khooyp@cs.umd.edu>, gdb-patches@sourceware.org
Subject: Re: Document how to exit "python-interactive"
Date: Thu, 23 Aug 2012 17:02:00 -0000	[thread overview]
Message-ID: <4DE79193-BBBE-46B5-854A-9CB70CC6B4FC@cs.umd.edu> (raw)
In-Reply-To: <83boi1a72i.fsf@gnu.org>

Hi,

On Aug 23, 2012, at 12:47 PM, Eli Zaretskii wrote:

>> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
>> --- a/gdb/doc/gdb.texinfo
>> +++ b/gdb/doc/gdb.texinfo
>> @@ -22515,7 +22515,8 @@
>> @item python-interactive @r{[}@var{command}@r{]}
>> @itemx pi @r{[}@var{command}@r{]}
>> Without an argument, the @code{python-interactive} command can be used
>> -to start an interactive Python prompt.
>> +to start an interactive Python prompt.  To return to @value{GDBN},
>> +type @code{EOF} (e.g., @samp{Ctrl-D}) on an empty prompt.
> 
> I think "type the @code{EOF} character" is better.
> 
> Also, please use @kbd{Ctrl-D}.  Finally, I think "on the empty prompt"
> is redundant (and may even be incorrect -- doesn't it work even on a
> non-empty line?).

Ctrl-D must be on an empty line to signal EOF, at least on Linux and probably other Unix systems. On Windows, EOF is supposed to be Ctrl-Z followed by enter, but I don't know if it needs to be on an empty line. I'll move the "on an empty prompt" into the parentheses to be more accurate.

>> 	   python_interactive_command,
>> #ifdef HAVE_PYTHON
>> 	   _("\
>> -Start a Python interactive prompt.\n\
>> +Start a Python interactive prompt.  To return to GDB, type EOF\n\
> 
> I think the second sentence should begin on a new line.  Don't you get
> that line truncated in "apropos" output?

I hadn't known about "apropos", but the help prints fine since "apropos" appears to print only up to the first period. I'll leave it as is.

If it sounds good, I'll make the changes and check it in.

Thanks!

Yit
August 23, 2012


  reply	other threads:[~2012-08-23 17:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-23 16:29 Khoo Yit Phang
2012-08-23 16:47 ` Eli Zaretskii
2012-08-23 17:02   ` Khoo Yit Phang [this message]
2012-08-23 17:47     ` Khoo Yit Phang
2012-08-23 18:04       ` Eli Zaretskii
2012-08-23 18:30         ` Khoo Yit Phang
2012-08-23 17:20 ` Pedro Alves
2012-08-23 17:39   ` Khoo Yit Phang
2012-08-23 17:55     ` Pedro Alves

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=4DE79193-BBBE-46B5-854A-9CB70CC6B4FC@cs.umd.edu \
    --to=khooyp@cs.umd.edu \
    --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