This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] NEWS: Mention "set foo unlimited".
- From: Pedro Alves <palves at redhat dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 16 Apr 2013 15:18:31 +0100
- Subject: Re: [PATCH] NEWS: Mention "set foo unlimited".
- References: <20130416112224 dot 5077 dot 74439 dot stgit at brno dot lan> <83sj2q4nfc dot fsf at gnu dot org>
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.