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

Eli Zaretskii
2011-11-03T16:57:43Z

> 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.