Hi Marcus,
Marcus Meissner wrote:
Hi,
sometime we have failure cases where we still return S_OK.
I am not sure this is a good idea.
NULL is a valid return value of these function, so it's fine to return S_OK here (also you've changed non-error code paths).
Thanks, Jacek