On Fri, Feb 18, 2011 at 01:22:41PM -0700, Eric Blake wrote:
> On 02/18/2011 12:30 PM, Jim Meyering wrote:
> >>> -if (doms->objs)
> >>> -virHashFree(doms->objs, virDomainObjListDeallocator);
> >>> +virHashFree(doms->objs, virDomainObjListDeallocator);
> >>
> >> I tried adding --name=
On 02/18/2011 12:30 PM, Jim Meyering wrote:
>>> -if (doms->objs)
>>> -virHashFree(doms->objs, virDomainObjListDeallocator);
>>> +virHashFree(doms->objs, virDomainObjListDeallocator);
>>
>> I tried adding --name=virHashFree to the useless_free_options variable
>> in cfg.mk, to see if
Eric Blake wrote:
> [adding Jim to cc, as author of useless-if-before-free]
>
> On 02/17/2011 02:14 PM, Christophe Fergeau wrote:
>> virHashFree follows the convention described in HACKING that
>> XXXFree() functions can be called with a NULL argument.
>> ---
>> src/conf/domain_conf.c |6 +---
[adding Jim to cc, as author of useless-if-before-free]
On 02/17/2011 02:14 PM, Christophe Fergeau wrote:
> virHashFree follows the convention described in HACKING that
> XXXFree() functions can be called with a NULL argument.
> ---
> src/conf/domain_conf.c |6 +---
> src/datatypes.c
virHashFree follows the convention described in HACKING that
XXXFree() functions can be called with a NULL argument.
---
src/conf/domain_conf.c |6 +---
src/datatypes.c | 51 +++---
src/qemu/qemu_process.c |4 +--
3 files changed, 20 inser