Re: [libvirt] [PATCH 07/26] virsh: Resolve Coverity DEADCODE

2014-09-11 Thread Peter Krempa
On 09/05/14 00:26, John Ferlan wrote: > Since 0766783abbe8bbc9ea686c2c3149f4c0ac139e19 > > Coverity complains that the EDIT_FREE definition results in DEADCODE. > > As it turns out with the change to use the EDIT_FREE macro the call to > vir*Free() wouldn't be necessary nor would it happen... >

[libvirt] [PATCH 07/26] virsh: Resolve Coverity DEADCODE

2014-09-04 Thread John Ferlan
Since 0766783abbe8bbc9ea686c2c3149f4c0ac139e19 Coverity complains that the EDIT_FREE definition results in DEADCODE. As it turns out with the change to use the EDIT_FREE macro the call to vir*Free() wouldn't be necessary nor would it happen... Prior code to above commitid would : vir*Ptr foo