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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



在 2016-05-14 00:00:18,"Andrea Bolognani" <abologna@xxxxxxxxxx> 写道:
>On Fri, 2016-05-13 at 16:19 +0100, Daniel P. Berrange wrote:
>> 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@xxxxxxx> wrote:
>> > > > 
>> > > >  
>> > > > From: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
>> > > >  
>> > > > Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
>> > > > ---
>> > > > 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.
>
>I guess that's a NACK then. Sorry, Chen! 
>

Hi, Andrea, Dan,

    Thanks for your clarification.

Regards,

-Chen

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]