Re: [libvirt] [PATCH] introduce VIR_CLOSE to be used rather than close()

>>> in a multi-threaded system, I am introducing the VIR_CLOSE(fd) macro >>> to help >>> avoid mistakes here. >>> VIR_FREE() is commonly used in the way that Matthias describes, so I think it would be consistent to use VIR_CLOSE in this way as well. Invoking it unconditionally simplifies the code, and eliminates possible confusion when someone adds new code and isn't sure which label they should goto on failure (or maybe they think they are sure, but make the wrong assumption, thus leading to a leaked fd as a result of trying to eliminate possible double-closed fd's ;-)