Re: usp10: Resend - Add funtionality for ScriptStringAnalyse

[Jeff Latimer]

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