Re: [PATCH] NEWS: Mention "set foo unlimited".

[Pedro Alves <palves at redhat dot com>]

On 04/16/2013 02:25 PM, Eli Zaretskii wrote:
>> From: Pedro Alves <palves@redhat.com>
>> Date: Tue, 16 Apr 2013 12:22:24 +0100
>>
>> I realized I forgot a NEWS entry for this.  I put it right below the
>> "New options" section.
>>
>> Okay?
> 
> OK, with one comment:
> 
>> +* Options that when set to 0 or -1 are handled specially as meaning
>> +  unlimited now also accept being set to literal "unlimited".  E.g.,
>> +  "set trace-buffer-size unlimited" is now an alias for "set
>> +  trace-buffer-size -1" and "set height unlimited" is now an alias for
>> +  "set height 0".
> 
> That first sentence took me 3 readings to understand.  How about the
> following wording instead:
> 
>   You can now use a literal value 'unlimited' for options that
>   interpret 0 or -1 as meaning "unlimited".

Great, I Love it, thanks.  Here's what I'm applying.

----------
Mention "set foo unlimited" in NEWS, right below the "New options"
section.

2013-04-16  Pedro Alves  <palves@redhat.com>
	    Eli Zaretskii  <eliz@gnu.org>

	* NEWS: Mention "set foo unlimited".
---

 gdb/NEWS |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/gdb/NEWS b/gdb/NEWS
index 86716e0..10a62f5 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -16,6 +16,12 @@ set remote trace-status-packet
 show remote trace-status-packet
   Set/show the use of remote protocol qTStatus packet.

+* You can now use a literal value 'unlimited' for options that
+  interpret 0 or -1 as meaning "unlimited".  E.g., "set
+  trace-buffer-size unlimited" is now an alias for "set
+  trace-buffer-size -1" and "set height unlimited" is now an alias for
+  "set height 0".
+
 * The command 'tsave' can now support new option '-ctf' to save trace
   buffer in Common Trace Format.