Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFA] add note on command naming to gdbint.texi
@ 2010-06-03 18:14 Doug Evans
  2010-06-03 18:44 ` Eli Zaretskii
  2010-06-03 19:36 ` Tom Tromey
  0 siblings, 2 replies; 4+ messages in thread
From: Doug Evans @ 2010-06-03 18:14 UTC (permalink / raw)
  To: gdb-patches

Hi.

I'd like to add this to the Coding section of gdbint.texi.

Ok?

2010-06-03  Doug Evans  <dje@google.com>

	* gdbint.texinfo (Coding): Add subsection on command names.

Index: doc/gdbint.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdbint.texinfo,v
retrieving revision 1.323
diff -u -p -r1.323 gdbint.texinfo
--- doc/gdbint.texinfo	2 May 2010 23:52:14 -0000	1.323
+++ doc/gdbint.texinfo	3 Jun 2010 18:05:31 -0000
@@ -6245,6 +6245,10 @@ the code detected a user error, recovere
 @code{warning} or the code failed to correctly recover from the user
 error and issued an @code{internal_error}.}
 
+@subsection Command Names
+
+GDB U/I commands are written @samp{foo-bar}, not @samp{foo_bar}.
+
 @subsection File Names
 
 Any file used when building the core of @value{GDBN} must be in lower


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

* Re: [RFA] add note on command naming to gdbint.texi
  2010-06-03 18:14 [RFA] add note on command naming to gdbint.texi Doug Evans
@ 2010-06-03 18:44 ` Eli Zaretskii
  2010-06-03 19:36 ` Tom Tromey
  1 sibling, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2010-06-03 18:44 UTC (permalink / raw)
  To: Doug Evans; +Cc: gdb-patches

> Date: Thu,  3 Jun 2010 11:13:53 -0700 (PDT)
> From: dje@google.com (Doug Evans)
> 
> I'd like to add this to the Coding section of gdbint.texi.
> 
> Ok?

Fine with me, thanks.


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

* Re: [RFA] add note on command naming to gdbint.texi
  2010-06-03 18:14 [RFA] add note on command naming to gdbint.texi Doug Evans
  2010-06-03 18:44 ` Eli Zaretskii
@ 2010-06-03 19:36 ` Tom Tromey
  2010-06-03 20:31   ` Stan Shebs
  1 sibling, 1 reply; 4+ messages in thread
From: Tom Tromey @ 2010-06-03 19:36 UTC (permalink / raw)
  To: Doug Evans; +Cc: gdb-patches

>>>>> "Doug" == Doug Evans <dje@google.com> writes:

Doug> I'd like to add this to the Coding section of gdbint.texi.

Doug> +GDB U/I commands are written @samp{foo-bar}, not @samp{foo_bar}.

I wonder whether we also have a rule about when to use a hyphen versus
when to use a subcommand.

I don't think that impacts whether this patch should go in.

Tom


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

* Re: [RFA] add note on command naming to gdbint.texi
  2010-06-03 19:36 ` Tom Tromey
@ 2010-06-03 20:31   ` Stan Shebs
  0 siblings, 0 replies; 4+ messages in thread
From: Stan Shebs @ 2010-06-03 20:31 UTC (permalink / raw)
  To: tromey; +Cc: Doug Evans, gdb-patches

Tom Tromey wrote:
>>>>>> "Doug" == Doug Evans <dje@google.com> writes:
>>>>>>             
>
> Doug> I'd like to add this to the Coding section of gdbint.texi.
>
> Doug> +GDB U/I commands are written @samp{foo-bar}, not @samp{foo_bar}.
>
> I wonder whether we also have a rule about when to use a hyphen versus
> when to use a subcommand.
>   

I think it's been semi-random to date (or is that "semi random"? :-) ).  
It would make sense to suggest using subcommands if there are going to 
be several related commands for which "foo" is a sensible name for the 
group.  It shouldn't be required though, since for instance the 
permissions patch I'm working on has commands "set may-write-registers" 
and "set may-insert-breakpoints", but "set may write-registers" is just 
going to be mystifying.

The suggestion should also mention the possibility of using "bar" as the 
common subcommand prefix, if the grouping is more logical or easier to 
remember that way.  For instance, I never used to remember whether it 
was "set remote-debug" or "set debug-remote", but "debug remote" is easy 
because now it's part of the debug group of set commands.

Stan


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

end of thread, other threads:[~2010-06-03 20:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-03 18:14 [RFA] add note on command naming to gdbint.texi Doug Evans
2010-06-03 18:44 ` Eli Zaretskii
2010-06-03 19:36 ` Tom Tromey
2010-06-03 20:31   ` Stan Shebs

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