On Tue, Nov 17, 2015 at 04:25:22PM +0100, Markus Armbruster wrote:
> I apologize for the lateness of my review.
> > +void object_property_iter_free(ObjectPropertyIterator *iter)
> > +{
> > + if (!iter) {
> > + return;
> > + }
> > + g_free(iter);
>
> g_free(NULL) is perfectly safe; please drop the conditional.
The next patch in the series has to free some fields in 'iter'
so the check for NULL upfront is not redundant.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
