Re: [Qemu-devel] [PATCH 0/1] Make the help info more friendly in monitor

2011-10-11 Thread Wayne Xia

于 2011-10-4 21:55, Luiz Capitulino 写道:

On Wed, 28 Sep 2011 10:16:19 +0100
Stefan Hajnoczistefa...@gmail.com  wrote:


On Wed, Sep 28, 2011 at 10:00 AM, Wayne Xiaxiaw...@linux.vnet.ibm.com  wrote:

During my test, I found it inconvenient when I type help or help info,
because the information was shown without orderliness. This patch would just
show the help information in sorted order.

For eg:
(qemu)help
acl_add
acl_policy
acl_remove
acl_reset
acl_show
balloon
block_passwd
...
the command list is sorted.

Wayne Xia (1):
  Sort the help info shown in monitor

  monitor.c |   97 ++--
  1 files changed, 93 insertions(+), 4 deletions(-)


This is a nice idea.  We could keep hmp/qmp-commands.hx in sorted
order but that prevents us from keeping related commands together in
those files (and the generated documentation?).  So sorting at
run-time makes sense.


The info help command reads from hmp-commands.hx and the info_cmds array,
I would prefer to get those sorted.


thanks, sorting info_cmds array seems more reasonable for that related
things would be kept here in one place not another queue, would change
in next patch.


--
Best Regards

Wayne Xia
mail:xiaw...@linux.vnet.ibm.com
tel:86-010-82450803




Re: [Qemu-devel] [PATCH 0/1] Make the help info more friendly in monitor

2011-10-04 Thread Luiz Capitulino
On Wed, 28 Sep 2011 10:16:19 +0100
Stefan Hajnoczi stefa...@gmail.com wrote:

 On Wed, Sep 28, 2011 at 10:00 AM, Wayne Xia xiaw...@linux.vnet.ibm.com 
 wrote:
  During my test, I found it inconvenient when I type help or help info,
  because the information was shown without orderliness. This patch would just
  show the help information in sorted order.
 
  For eg:
  (qemu)help
  acl_add
  acl_policy
  acl_remove
  acl_reset
  acl_show
  balloon
  block_passwd
  ...
  the command list is sorted.
 
  Wayne Xia (1):
   Sort the help info shown in monitor
 
   monitor.c |   97 
  ++--
   1 files changed, 93 insertions(+), 4 deletions(-)
 
 This is a nice idea.  We could keep hmp/qmp-commands.hx in sorted
 order but that prevents us from keeping related commands together in
 those files (and the generated documentation?).  So sorting at
 run-time makes sense.

The info help command reads from hmp-commands.hx and the info_cmds array,
I would prefer to get those sorted.



[Qemu-devel] [PATCH 0/1] Make the help info more friendly in monitor

2011-09-28 Thread Wayne Xia
During my test, I found it inconvenient when I type help or help info,
because the information was shown without orderliness. This patch would just
show the help information in sorted order.

For eg:
(qemu)help
acl_add
acl_policy
acl_remove
acl_reset
acl_show
balloon
block_passwd
...
the command list is sorted.

Wayne Xia (1):
  Sort the help info shown in monitor

 monitor.c |   97 ++--
 1 files changed, 93 insertions(+), 4 deletions(-)

-- 
1.7.6





Re: [Qemu-devel] [PATCH 0/1] Make the help info more friendly in monitor

2011-09-28 Thread Stefan Hajnoczi
On Wed, Sep 28, 2011 at 10:00 AM, Wayne Xia xiaw...@linux.vnet.ibm.com wrote:
 During my test, I found it inconvenient when I type help or help info,
 because the information was shown without orderliness. This patch would just
 show the help information in sorted order.

 For eg:
 (qemu)help
 acl_add
 acl_policy
 acl_remove
 acl_reset
 acl_show
 balloon
 block_passwd
 ...
 the command list is sorted.

 Wayne Xia (1):
  Sort the help info shown in monitor

  monitor.c |   97 ++--
  1 files changed, 93 insertions(+), 4 deletions(-)

This is a nice idea.  We could keep hmp/qmp-commands.hx in sorted
order but that prevents us from keeping related commands together in
those files (and the generated documentation?).  So sorting at
run-time makes sense.

Stefan