Hi Joerg,

> Am I worrying too much or should any such piece of code be protected by
> TRACE("=> 1/ %x (%s)\n", dwRet, LOWORD(dwRet) ? "" : debugstr_w(lpstrRet));
> (actually, an even more convoluted test is needed because of 
> MCI_INTEGER_RETURNED)

I'd worry about it.  We've sometimes had bugs where a crash appears in
a failure mode when a particular debug channel is enabled, but that
debug channel is the very thing that might help us debug the failure
better.

> I assume that debugstr_w is not executed when that channel is not traced. 
> Correct?

That's correct.
--Juan


Reply via email to