Wenchao Xia <xiaw...@linux.vnet.ibm.com> writes: > This patch make parsing of hmp command aware of that it may > have sub command. Also discard simple encapsulation function > monitor_find_command() and qmp_find_cmd(). > > Signed-off-by: Wenchao Xia <xiaw...@linux.vnet.ibm.com> > --- > monitor.c | 31 +++++++++++++++++++++++-------- > 1 files changed, 23 insertions(+), 8 deletions(-) > > diff --git a/monitor.c b/monitor.c > index c7b3014..d49a362 100644 > --- a/monitor.c > +++ b/monitor.c > @@ -130,6 +130,7 @@ typedef struct mon_cmd_t { > MonitorCompletion *cb, void *opaque); > } mhandler; > int flags; > + struct mon_cmd_t *sub_table; > } mon_cmd_t; > > /* file descriptors passed via SCM_RIGHTS */
Forgot to mention: please explain in a comment how mhandler and sub_table interact. Namely, if there are arguments, we use sub_table, else mhandler.