On 09/27, Maxime Coquelin wrote: [snip] >> + devcnt = 0; >This isn't needed as it is static. > >> + interactive = 0; >Ditto > >> + memset(iface, 0, sizeof(iface)); >Ditto > >> + memset(vports, 0, sizeof(vports)); >And this one too when it will be made static. > > >Appart these small comments, it looks good to me. >When fixed, please apply my: > >Reviewed-by: Maxime coquelin <maxime.coque...@redhat.com> >
Thanks for your comments, I'll adpot all the static related suggetions. Thanks, Xiaolong >Thanks, >Maxime
