This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] `info proc *' help fix [Re: [patch] `info proc ' completion]
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: pedro at codesourcery dot com, gdb-patches at sourceware dot org, sergiodj at redhat dot com
- Date: Thu, 03 Nov 2011 18:56:48 +0200
- Subject: Re: [patch] `info proc *' help fix [Re: [patch] `info proc ' completion]
- References: <m3wrbr8olq.fsf@redhat.com> <m3vcr5u9y4.fsf@redhat.com> <20111031070012.GA32610@host1.jankratochvil.net> <201110311220.53564.pedro@codesourcery.com> <20111101115405.GA15861@host1.jankratochvil.net> <83pqhbn771.fsf@gnu.org> <20111103141144.GA20181@host1.jankratochvil.net>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Thu, 3 Nov 2011 15:11:44 +0100
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> Cc: pedro@codesourcery.com, gdb-patches@sourceware.org, sergiodj@redhat.com
>
> On Tue, 01 Nov 2011 17:22:42 +0100, Eli Zaretskii wrote:
> > I wonder if we could clean up this mess with 5 "different bunches of
> > info", while we are at that.
>
> Sorry I somehow mishandled your mail, OK this way?
It's better.
> add_cmd ("stat", class_info, linux_nat_info_proc_cmd_stat, _("\
> -List a bunch of random process info."),
> +List annotated random process info from /proc/PID/stat."),
> &info_proc_cmdlist);
I would suggest
List process info from /proc/PID/stat.
> -List a different bunch of random process info."),
> +List a different bunch of random process info from /proc/PID/status."),
And likewise here.
> -List a different bunch of random process info."),
> +List current working directory of the process."),
> &info_proc_cmdlist);
>
> add_cmd ("cmdline", class_info, linux_nat_info_proc_cmd_cmdline, _("\
> -List a different bunch of random process info."),
> +List command line arguments of the process."),
> &info_proc_cmdlist);
>
> add_cmd ("exe", class_info, linux_nat_info_proc_cmd_exe, _("\
> -List a different bunch of random process info."),
> +List absolute filename for executable of the process."),
> &info_proc_cmdlist);
These are OK.
Thanks.