Re: [PATCH 3/4] d3dx9: Implement ID3DXBaseEffect::GetPixelShader().

[Henri Verbeet]

2011/5/3 Rico Schüller <kgbricola(a)web.de>:
> +#define SAFE_ADDREF(x) if(x) IUnknown_AddRef(x)
> +#define SAFE_RELEASE(x) if(x) IUnknown_Release(x)
> That's not so safe, actually. Consider e.g. what happens when "x" has
> side effects, or when you write something like "if (cond)
> SAFE_ADDREF(x); else return;". If you really want this an inline
> function should do just as well, but for the handful of uses I see in
> this series it just obfuscates the code.