Re: [libvirt] [PATCH] qemu-agent: remove redundant comments

On Fri, May 13, 2016 at 05:17:35PM +0200, Andrea Bolognani wrote: > > On Fri, 2016-05-13 at 17:16 +0800, Chen Hanxiao wrote: > > > > At 2016-05-10 15:17:48, "Chen Hanxiao" <chen_han_xiao(a)126.com&gt; wrote: > > > > > >   > > > From: Chen Hanxiao <chenhanxiao(a)gmail.com&gt; > > >   > > > Signed-off-by: Chen Hanxiao <chenhanxiao(a)gmail.com&gt; > > > --- > > > src/qemu/qemu_agent.c | 11 ----------- > > > 1 file changed, 11 deletions(-) > > >   > > > diff --git a/src/qemu/qemu_agent.c b/src/qemu/qemu_agent.c > > > index c55f304..20597b4 100644 > > > --- a/src/qemu/qemu_agent.c > > > +++ b/src/qemu/qemu_agent.c > > > @@ -287,17 +287,6 @@ qemuAgentIOProcessEvent(qemuAgentPtr mon, > > >          return -1; > > >      } > > >   > > > -/* > > > -    for (i = 0; i < ARRAY_CARDINALITY(eventHandlers); i++) { > > > -        if (STREQ(eventHandlers[i].type, type)) { > > > -            virJSONValuePtr data = virJSONValueObjectGet(obj, "data"); > > > -            VIR_DEBUG("handle %s handler=%p data=%p", type, > > > -                      eventHandlers[i].handler, data); > > > -            (eventHandlers[i].handler)(mon, data); > > > -            break; > > > -        } > > > -    } > > > -*/ > > >      return 0; > > > } > > >   > This chunk is referenced near the top of the file: > > > > > /* When you are the first to uncomment this, > >   * don't forget to uncomment the corresponding > >   * part in qemuAgentIOProcessEvent as well. > >   * > > static struct { > >      const char *type; > >      void (*handler)(qemuAgentPtr mon, virJSONValuePtr data); > > } eventHandlers[] = { > > }; > > */ > > Looks like we haven't felt the need to enable it in > about five years, so maybe we can just get rid of it. > > CC'ing Dan, who wrote the code. I only wrote the monitor code, not the agent code. This just looks like it was copied from the monitor code to prepare us for a future where the agent starts sending events. Personally I'd just leave all this code as-is.