8 Apr
2008
8 Apr
'08
11:22 a.m.
Dan Hipschman <dsh(a)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. -- Alexandre Julliard julliard(a)winehq.org