Dan Hipschman dsh@linux.ucla.edu writes:
+#define RELEASE(iface) \
- do \
if (iface) \
{ \
IUnknown_Release((IUnknown *) iface); \
iface = NULL; \
} \
- while (0)
Please avoid that sort of thing, it's ugly to modify what looks like an input argument inside a macro. Besides, you really don't need a macro for two lines of code.