[ovs-dev] [PATCH 1/2] netdev-windows: fix ofpbuf initialization

2016-10-27 Thread Alin Serdean
ofpbuf_const_initializer takes only two parameters see: https://github.com/openvswitch/ovs/blob/master/include/openvswitch/ofpbuf.h#L96 the compiler only gives the following warning: lib/netdev-windows.c(229) : warning C4020: 'ofpbuf_const_initializer' : too many actual parameters which in resu

Re: [ovs-dev] [PATCH 1/2] netdev-windows: fix ofpbuf initialization

2016-10-27 Thread Sairam Venugopal
I thought we had set warnings to be treated as error. Appveyor didn¹t catch this one either. Thanks for fixing this. Acked-by: Sairam Venugopal On 10/27/16, 12:20 PM, "Alin Serdean" wrote: >ofpbuf_const_initializer takes only two parameters see: > >https://urldefense.proofpoint.com/v2/url?u=h

Re: [ovs-dev] [PATCH 1/2] netdev-windows: fix ofpbuf initialization

2016-11-03 Thread Sairam Venugopal
Guru, Any reason why the acked-by doesn’t show up on Patchwork - https://patchwork.ozlabs.org/patch/687913/ Thanks, Sairam On 10/27/16, 1:49 PM, "Sairam Venugopal" wrote: >I thought we had set warnings to be treated as error. Appveyor didn¹t >catch this one either. Thanks for fixing this. > >A

Re: [ovs-dev] [PATCH 1/2] netdev-windows: fix ofpbuf initialization

2016-11-04 Thread Guru Shetty
On 3 November 2016 at 16:25, Sairam Venugopal wrote: > Guru, > > Any reason why the acked-by doesn’t show up on Patchwork - > https://patchwork.ozlabs.org/patch/687913/ Not sure. Just flaky maybe. I applied this series. Thanks. > > > Thanks, > Sairam > > On 10/27/16, 1:49 PM, "Sairam Venugopa