Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [commit] Doc: -var-delete
@ 2006-12-04  3:57 Nick Roberts
  2006-12-04  4:10 ` Eli Zaretskii
  2006-12-04  9:16 ` [commit] Doc: -var-delete Vladimir Prus
  0 siblings, 2 replies; 9+ messages in thread
From: Nick Roberts @ 2006-12-04  3:57 UTC (permalink / raw)
  To: gdb-patches


Committed as obvious.

-- 
Nick                                           http://www.inet.net.nz/~nickrob


2006-12-04  Nick Roberts  <nickrob@snap.net.nz>

	* gdb.texinfo (GDB/MI Variable Objects): Describe -c option
	of -var-delete.


*** gdb.texinfo	23 Nov 2006 09:27:02 +1300	1.371
--- gdb.texinfo	04 Dec 2006 16:45:48 +1300	
***************
*** 19598,19604 ****
  @item @code{-var-create}
  @tab create a variable object
  @item @code{-var-delete}
! @tab delete the variable object and its children
  @item @code{-var-set-format}
  @tab set the display format of this variable
  @item @code{-var-show-format}
--- 19598,19604 ----
  @item @code{-var-create}
  @tab create a variable object
  @item @code{-var-delete}
! @tab delete the variable object and/or its children
  @item @code{-var-set-format}
  @tab set the display format of this variable
  @item @code{-var-show-format}
***************
*** 19681,19690 ****
  @subsubheading Synopsis
  
  @smallexample
!  -var-delete @var{name}
  @end smallexample
  
  Deletes a previously created variable object and all of its children.
  
  Returns an error if the object @var{name} is not found.
  
--- 19681,19691 ----
  @subsubheading Synopsis
  
  @smallexample
!  -var-delete [ -c ] @var{name}
  @end smallexample
  
  Deletes a previously created variable object and all of its children.
+ With the @samp{-c} option, just deletes the children.
  
  Returns an error if the object @var{name} is not found.
  


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

* Re: [commit] Doc: -var-delete
  2006-12-04  3:57 [commit] Doc: -var-delete Nick Roberts
@ 2006-12-04  4:10 ` Eli Zaretskii
  2006-12-04  4:45   ` Nick Roberts
  2006-12-04  9:16 ` [commit] Doc: -var-delete Vladimir Prus
  1 sibling, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2006-12-04  4:10 UTC (permalink / raw)
  To: Nick Roberts; +Cc: gdb-patches

> From: Nick Roberts <nickrob@snap.net.nz>
> Date: Mon, 4 Dec 2006 16:52:51 +1300
> 
> Committed as obvious.

Not that I object to the change, but I'm curious: why did you think it
was obvious?


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

* Re: [commit] Doc: -var-delete
  2006-12-04  4:10 ` Eli Zaretskii
@ 2006-12-04  4:45   ` Nick Roberts
  2006-12-04 19:45     ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Nick Roberts @ 2006-12-04  4:45 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gdb-patches

 > Not that I object to the change, but I'm curious: why did you think it
 > was obvious?

I guess if it really was obvious then someone would have made the change before
me but I was trying to apply the definition in MAINTAINERS:

  An "obvious fix" means that there is no possibility that anyone will
  disagree with the change.

As I'm documenting existing (undocumented) functionality I can't see how anyone
can disagree.  You might be able to improve upon it, but that's presumably a
different point and you are still able to do so.

Why do you think it is not obvious?

-- 
Nick                                           http://www.inet.net.nz/~nickrob


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

* Re: [commit] Doc: -var-delete
  2006-12-04  3:57 [commit] Doc: -var-delete Nick Roberts
  2006-12-04  4:10 ` Eli Zaretskii
@ 2006-12-04  9:16 ` Vladimir Prus
  1 sibling, 0 replies; 9+ messages in thread
From: Vladimir Prus @ 2006-12-04  9:16 UTC (permalink / raw)
  To: gdb-patches

Nick Roberts wrote:

> @smallexample
> !  -var-delete [ -c ] @var{name}
> @end smallexample
> 
> Deletes a previously created variable object and all of its children.
> + With the @samp{-c} option, just deletes the children.

Should this be "deletes just the children"? Or "deletes only the children"?
The way it's written now "just" binds to "deletes", so one wonders if
there's "just delete" behaviour and "not just delete" behaviour and what
the latter might be.

non-native-speakerly-picky-yours,
Volodya







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

* Re: [commit] Doc: -var-delete
  2006-12-04  4:45   ` Nick Roberts
@ 2006-12-04 19:45     ` Eli Zaretskii
  2006-12-04 20:57       ` Nick Roberts
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2006-12-04 19:45 UTC (permalink / raw)
  To: Nick Roberts; +Cc: gdb-patches

> From: Nick Roberts <nickrob@snap.net.nz>
> Date: Mon, 4 Dec 2006 17:41:03 +1300
> Cc: gdb-patches@sources.redhat.com
> 
> I was trying to apply the definition in MAINTAINERS:
> 
>   An "obvious fix" means that there is no possibility that anyone will
>   disagree with the change.
> 
> As I'm documenting existing (undocumented) functionality I can't see how anyone
> can disagree.

The _need_ for documenting an existing feature is indisputable (at
least in this case, since that is my policy; you know as well as I do
that, in the case of Emacs, for example, RMS has a different policy,
so even the need for documenting an existing feature does not
necessarily constitute a good-enough reason).  However, _how_ it
should be documented is not obvious in any way; I could express the
same idea as you did in about a dozen different ways.

> Why do you think it is not obvious?

Because about the only kind of fix in the docs that is really obvious
is a typo or bad English.


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

* Re: [commit] Doc: -var-delete
  2006-12-04 19:45     ` Eli Zaretskii
@ 2006-12-04 20:57       ` Nick Roberts
  2006-12-08 17:29         ` Eli Zaretskii
  2006-12-08 17:32         ` [commit] Clarify obvious fix in the docs (was: Doc: -var-delete) Eli Zaretskii
  0 siblings, 2 replies; 9+ messages in thread
From: Nick Roberts @ 2006-12-04 20:57 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gdb-patches

 > > Why do you think it is not obvious?
 > 
 > Because about the only kind of fix in the docs that is really obvious
 > is a typo or bad English.

OK, sorry.  I'll post to gdb-patches first, next time.  To avoid any confusion,
it might be a good idea to say this in MAINTAINERS.

-- 
Nick                                           http://www.inet.net.nz/~nickrob


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

* Re: [commit] Doc: -var-delete
  2006-12-04 20:57       ` Nick Roberts
@ 2006-12-08 17:29         ` Eli Zaretskii
  2006-12-08 17:32         ` [commit] Clarify obvious fix in the docs (was: Doc: -var-delete) Eli Zaretskii
  1 sibling, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2006-12-08 17:29 UTC (permalink / raw)
  To: Nick Roberts; +Cc: gdb-patches

> From: Nick Roberts <nickrob@snap.net.nz>
> Date: Tue, 5 Dec 2006 09:52:30 +1300
> Cc: gdb-patches@sources.redhat.com
> 
>  > > Why do you think it is not obvious?
>  > 
>  > Because about the only kind of fix in the docs that is really obvious
>  > is a typo or bad English.
> 
> OK, sorry.  I'll post to gdb-patches first, next time.  To avoid any confusion,
> it might be a good idea to say this in MAINTAINERS.

Done; thanks for the suggestion.


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

* Re: [commit] Clarify obvious fix in the docs (was: Doc: -var-delete)
  2006-12-04 20:57       ` Nick Roberts
  2006-12-08 17:29         ` Eli Zaretskii
@ 2006-12-08 17:32         ` Eli Zaretskii
  1 sibling, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2006-12-08 17:32 UTC (permalink / raw)
  To: gdb-patches

Here's the patch I committed to clarify the ``obvious fix'' in the
docs:

2006-12-08  Eli Zaretskii  <eliz@gnu.org>

	* MAINTAINERS: Clarify the meaning of an ``obvious fix'' in the
	docs.

Index: MAINTAINERS
===================================================================
RCS file: /cvs/src/src/gdb/MAINTAINERS,v
retrieving revision 1.354
retrieving revision 1.355
diff -u -r1.354 -r1.355
--- MAINTAINERS	6 Dec 2006 06:51:50 -0000	1.354
+++ MAINTAINERS	8 Dec 2006 17:28:31 -0000	1.355
@@ -95,6 +95,9 @@
 fix, since such a change without discussion will result in
 instantaneous and loud complaints.
 
+For documentation changes, about the only kind of fix that is obvious
+is correction of a typo or bad English usage.
+
 
 	     GDB Steering Committee
 	     ----------------------


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

* Re: [commit] Doc: -var-delete
@ 2006-12-04 10:29 Nick Roberts
  0 siblings, 0 replies; 9+ messages in thread
From: Nick Roberts @ 2006-12-04 10:29 UTC (permalink / raw)
  To: gdb-patches; +Cc: Vladimir Prus


> > @smallexample
> > ! Â-var-delete [ -c ] @var{name}
> > @end smallexample
> > 
> > Deletes a previously created variable object and all of its children.
> > + With the @samp{-c} option, just deletes the children.

> Should this be "deletes just the children"? Or "deletes only the children"?
> The way it's written now "just" binds to "deletes", so one wonders if
> there's "just delete" behaviour and "not just delete" behaviour and what
> the latter might be.

If it were a legal contract, I think you may have found a loophole.  For
general reading though, I think the second sentence is unambiguous in the
context of the first.  However I don't mind if someone wants to change it.

-- 
Nick http://www.inet.net.nz/~nickrob


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

end of thread, other threads:[~2006-12-08 17:32 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-12-04  3:57 [commit] Doc: -var-delete Nick Roberts
2006-12-04  4:10 ` Eli Zaretskii
2006-12-04  4:45   ` Nick Roberts
2006-12-04 19:45     ` Eli Zaretskii
2006-12-04 20:57       ` Nick Roberts
2006-12-08 17:29         ` Eli Zaretskii
2006-12-08 17:32         ` [commit] Clarify obvious fix in the docs (was: Doc: -var-delete) Eli Zaretskii
2006-12-04  9:16 ` [commit] Doc: -var-delete Vladimir Prus
2006-12-04 10:29 Nick Roberts

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