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