Re: [libvirt] [DRAFT PATCHv5 9/9] rpc: removes the removal of unused variable i

On 12/20/2011 01:35 AM, Hu Tao wrote: Oh my. That certainly explains things. But removing this line re-introduces a compiler warning about an unused variable 'i', so a better fix would be restricting matches to non-member uses of 'buf' and 'i': diff --git i/src/rpc/genprotocol.pl w/src/rpc/genprotocol.pl index 7af1b3b..4838325 100755 --- i/src/rpc/genprotocol.pl +++ w/src/rpc/genprotocol.pl @@ -67,12 +67,12 @@ while (<RPCGEN>) { # Note: The body of the function is in @function. # Remove decl of buf, if buf isn't used in the function. - my @uses = grep /\bbuf\b/, @function; - @function = grep !/\bbuf\b/, @function if @uses == 1; + my @uses = grep /[^.>]\bbuf\b/, @function; + @function = grep !/[^.>]\bbuf\b/, @function if @uses == 1; # Remove decl of i, if i isn't used in the function. - @uses = grep /\bi\b/, @function; - @function = grep !/\bi\b/, @function if @uses == 1; + @uses = grep /[^.>]\bi\b/, @function; + @function = grep !/[^.>]\bi\b/, @function if @uses == 1; # (char **)&objp->... gives: # warning: dereferencing type-punned pointer will break -- Eric Blake eblake(a)redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org