[6/6] NEWS: "info" commands now list in ascending order

Message ID 1445507944-9197-7-git-send-email-palves@redhat.com
State New, archived
Headers

Commit Message

Pedro Alves Oct. 22, 2015, 9:59 a.m. UTC
  gdb/ChangeLog:
2015-10-22  Pedro Alves  <palves@redhat.com>

	* NEWS: Mention that a few "info" commands now list the
	corresponding items in ascending order.
---
 gdb/NEWS | 4 ++++
 1 file changed, 4 insertions(+)
  

Comments

Eli Zaretskii Oct. 22, 2015, 3 p.m. UTC | #1
> From: Pedro Alves <palves@redhat.com>
> Date: Thu, 22 Oct 2015 10:59:04 +0100
> 
> gdb/ChangeLog:
> 2015-10-22  Pedro Alves  <palves@redhat.com>
> 
> 	* NEWS: Mention that a few "info" commands now list the
> 	corresponding items in ascending order.
> ---
>  gdb/NEWS | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/gdb/NEWS b/gdb/NEWS
> index b2b1e99..e16e861 100644
> --- a/gdb/NEWS
> +++ b/gdb/NEWS
> @@ -24,6 +24,10 @@
>  
>  * GDB now supports displaced stepping on AArch64 GNU/Linux.
>  
> +* "info threads", "info inferiors", "info display", "info checkpoints"
> +  and "maint info program-spaces" now list the corresponding items in
> +  ascending order, for consistency with all other "info" commands.
             ^^
I'd suggest to add "alphabetic" where indicated.

Otherwise, this is fine, thanks.
  
Pedro Alves Oct. 22, 2015, 3:07 p.m. UTC | #2
On 10/22/2015 04:00 PM, Eli Zaretskii wrote:
>> From: Pedro Alves <palves@redhat.com>
>> Date: Thu, 22 Oct 2015 10:59:04 +0100
>>
>> gdb/ChangeLog:
>> 2015-10-22  Pedro Alves  <palves@redhat.com>
>>
>> 	* NEWS: Mention that a few "info" commands now list the
>> 	corresponding items in ascending order.
>> ---
>>  gdb/NEWS | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/gdb/NEWS b/gdb/NEWS
>> index b2b1e99..e16e861 100644
>> --- a/gdb/NEWS
>> +++ b/gdb/NEWS
>> @@ -24,6 +24,10 @@
>>  
>>  * GDB now supports displaced stepping on AArch64 GNU/Linux.
>>  
>> +* "info threads", "info inferiors", "info display", "info checkpoints"
>> +  and "maint info program-spaces" now list the corresponding items in
>> +  ascending order, for consistency with all other "info" commands.
>              ^^
> I'd suggest to add "alphabetic" where indicated.

That would sound very confusing to me, as we sort by e.g., thread
ID, not thread name, and alphabetic would suggest to me that we
now sort by "name" or whatever else looks like a name.

Would "ascending ID order" or "ascending numerical order"
instead be OK with you?

> 
> Otherwise, this is fine, thanks.
> 


Thanks,
Pedro Alves
  
Eli Zaretskii Oct. 22, 2015, 3:27 p.m. UTC | #3
> Date: Thu, 22 Oct 2015 16:07:30 +0100
> From: Pedro Alves <palves@redhat.com>
> CC: gdb-patches@sourceware.org
> 
> > I'd suggest to add "alphabetic" where indicated.
> 
> That would sound very confusing to me, as we sort by e.g., thread
> ID, not thread name, and alphabetic would suggest to me that we
> now sort by "name" or whatever else looks like a name.
> 
> Would "ascending ID order" or "ascending numerical order"
> instead be OK with you?

Yes, of course.  The point is just "ascending" doesn't say enough, we
should say more about the order.  (I mistakenly thought it was
alphabetic.)

Thanks.
  

Patch

diff --git a/gdb/NEWS b/gdb/NEWS
index b2b1e99..e16e861 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -24,6 +24,10 @@ 
 
 * GDB now supports displaced stepping on AArch64 GNU/Linux.
 
+* "info threads", "info inferiors", "info display", "info checkpoints"
+  and "maint info program-spaces" now list the corresponding items in
+  ascending order, for consistency with all other "info" commands.
+
 * New commands
 
 maint set target-non-stop (on|off|auto)