This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]