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
