Jeff L <[EMAIL PROTECTED]> writes:

> +    cMaxItems = 255;                 /* Pick an abritary size for buffers */
> +    if  (!*pssa)                     /* Clean it out and start again  */
> +        ScriptStringFree(pssa);

I still don't see why you want to free a NULL pointer.

> +    {
> +        prtssa(*pssa);
> +        analysis = * pssa;
> +        prtssa((SCRIPT_STRING_ANALYSIS *)analysis);

Why do you print the same object twice?

There seems to be a lot of confusion in your code between pointers and
pointers to pointers, I think you need to define better data types to
clarify this.

-- 
Alexandre Julliard
[EMAIL PROTECTED]


Reply via email to