Re: [patch] `info proc *' help fix [Re: [patch] `info proc ' completion]

Jan Kratochvil
2011-11-03T17:07:00Z

On Thu, 03 Nov 2011 17:56:48 +0100, Eli Zaretskii wrote:
> > 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.

The problem is each `info proc XXX' commands reads it in /proc/PID/XXX file,
which may be obvious. But I understand "stat" and "status" may be an
exception as it is difficult to describe what /proc/PID/{stat,status} contain.

I assume you meant it this way but asking again.

Thanks,
Jan

gdb/
2011-11-03 Jan Kratochvil <jan.kratochvil@redhat.com>
Eli Zaretskii <eliz@gnu.org>

* linux-nat.c (_initialize_linux_nat): Improve help
for `info proc stat', `info proc status', `info proc cwd',
`info proc cmdline' and `info proc exe'.

--- a/gdb/linux-nat.c
+++ b/gdb/linux-nat.c
@@ -5887,23 +5887,23 @@ List of mapped memory regions."),
&info_proc_cmdlist);

add_cmd ("stat", class_info, linux_nat_info_proc_cmd_stat, _("\
-List a bunch of random process info."),
+List process info from /proc/PID/stat."),
&info_proc_cmdlist);

add_cmd ("status", class_info, linux_nat_info_proc_cmd_status, _("\
-List a different bunch of random process info."),
+List process info from /proc/PID/status."),
&info_proc_cmdlist);

add_cmd ("cwd", class_info, linux_nat_info_proc_cmd_cwd, _("\
-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);

add_cmd ("all", class_info, linux_nat_info_proc_cmd_all, _("\