Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFA] Improve on-line help for thread_apply_command and thread_apply_all_command.
@ 2018-04-20 21:19 Philippe Waroquiers
  2018-04-20 21:22 ` Pedro Alves
  0 siblings, 1 reply; 3+ messages in thread
From: Philippe Waroquiers @ 2018-04-20 21:19 UTC (permalink / raw)
  To: gdb-patches; +Cc: Philippe Waroquiers

Add a Usage: line for thread_apply_command, in particular to mention
the thread ID list.

In thread_apply_command and thread_apply_all_command help, use
uppercase for arg names, as this style seems to be more standard.

2018-04-20  Philippe Waroquiers  <philippe.waroquiers@skynet.be>

	* thread.c (_initialize_thread): improve on-line help for
	thread_apply_command and thread_apply_all_command.
---
 gdb/thread.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gdb/thread.c b/gdb/thread.c
index 23aa5ba117..a09d7e0ba0 100644
--- a/gdb/thread.c
+++ b/gdb/thread.c
@@ -2028,15 +2028,17 @@ The new thread ID must be currently known."),
 		  &thread_cmd_list, "thread ", 1, &cmdlist);
 
   add_prefix_cmd ("apply", class_run, thread_apply_command,
-		  _("Apply a command to a list of threads."),
+		  _("Apply a command to a list of threads.\n\
+Usage: thread apply ID... COMMAND\n\
+ID is a space-separated list of IDs of threads to apply COMMAND on."),
 		  &thread_apply_list, "thread apply ", 1, &thread_cmd_list);
 
   add_cmd ("all", class_run, thread_apply_all_command,
 	   _("\
 Apply a command to all threads.\n\
 \n\
-Usage: thread apply all [-ascending] <command>\n\
--ascending: Call <command> for all threads in ascending order.\n\
+Usage: thread apply all [-ascending] COMMAND\n\
+-ascending: Call COMMAND for all threads in ascending order.\n\
             The default is descending order.\
 "),
 	   &thread_apply_list);
-- 
2.11.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFA] Improve on-line help for thread_apply_command and thread_apply_all_command.
  2018-04-20 21:19 [RFA] Improve on-line help for thread_apply_command and thread_apply_all_command Philippe Waroquiers
@ 2018-04-20 21:22 ` Pedro Alves
  2018-04-20 21:27   ` Philippe Waroquiers
  0 siblings, 1 reply; 3+ messages in thread
From: Pedro Alves @ 2018-04-20 21:22 UTC (permalink / raw)
  To: Philippe Waroquiers, gdb-patches

On 04/20/2018 10:19 PM, Philippe Waroquiers wrote:
> Add a Usage: line for thread_apply_command, in particular to mention
> the thread ID list.
> 
> In thread_apply_command and thread_apply_all_command help, use
> uppercase for arg names, as this style seems to be more standard.
> 
> 2018-04-20  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> 
> 	* thread.c (_initialize_thread): improve on-line help for
> 	thread_apply_command and thread_apply_all_command.

OK.

Thanks,
Pedro Alves


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFA] Improve on-line help for thread_apply_command and thread_apply_all_command.
  2018-04-20 21:22 ` Pedro Alves
@ 2018-04-20 21:27   ` Philippe Waroquiers
  0 siblings, 0 replies; 3+ messages in thread
From: Philippe Waroquiers @ 2018-04-20 21:27 UTC (permalink / raw)
  To: Pedro Alves, gdb-patches

On Fri, 2018-04-20 at 22:22 +0100, Pedro Alves wrote:
> On 04/20/2018 10:19 PM, Philippe Waroquiers wrote:
> > Add a Usage: line for thread_apply_command, in particular to mention
> > the thread ID list.
> > 
> > In thread_apply_command and thread_apply_all_command help, use
> > uppercase for arg names, as this style seems to be more standard.
> > 
> > 2018-04-20  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> > 
> > 	* thread.c (_initialize_thread): improve on-line help for
> > 	thread_apply_command and thread_apply_all_command.
> 
> OK.
Thanks, pushed.
Philippe


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-04-20 21:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-20 21:19 [RFA] Improve on-line help for thread_apply_command and thread_apply_all_command Philippe Waroquiers
2018-04-20 21:22 ` Pedro Alves
2018-04-20 21:27   ` Philippe Waroquiers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox