On Tue, Nov 13, 2012 at 6:30 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > When GLib deprecated g_mutex_new/g_mutex_free, we introduced a > compatibility wrapper to implement these using non-deprecated > functions. This was done by allocating 0-filled memory by the > mutex, and then letting GLib initialize the structure when > needed. However, we must call g_mutex_clear when destroying > the mutex to free these resources, which this patch fix. I didn't ACK this patch before because I wasn't sure if 'inline' functions is something we want to use here. Since we dont use them anywhere else in this codebase, I'd recommend you put these in the .c file as normal functions. No strong feelings either way so ACK with or without my suggestion acted on. -- Regards, Zeeshan Ali (Khattak) FSF member#5124 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list