Vitaliy Margolen wrote:

This one implements the checks described in the API. I think that we need to reproduce those checks but I can make the message more clear though.

Jeff

Can you use something that every other person would understand?
   if  (cString < 1 || !pString) {
Please use WARN if you want to indicate app's errors:
       WARN("cString %d, pString(%p)\n", cString, pString);
But in particular case, I don't see a need for this. You already have
all dumping TRACE right at the beginning of the function. It will log
any errors as well.

Vitaliy Margolen






Reply via email to