[1/2] qedit: Implement IMediaDet_put_Filename.

Alexandre Julliard julliard at winehq.org
Tue Apr 8 06:22:12 CDT 2008


Dan Hipschman <dsh at 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 at winehq.org



More information about the wine-devel mailing list