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

Jan Kratochvil jan.kratochvil@redhat.com
Thu Nov 3 17:07:00 GMT 2011


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, _("\



More information about the Gdb-patches mailing list