In "pdf-types.h" I see at least one error (if you test before calling
"pdf_i64_div" that divisor isn't 0, like in "pdf_i64_mod"):
>#define pdf_i64_div(dest, dividend, divisor, p_status) \
>  do                                                   \
>    {                                                  \
>      if (dest == NULL)                                \
>        {                                              \
>          *p_status =  PDF_ERROR;                      \
>        }                                              \
>      else                                             \
>        {                                              \
>          *p_status =  PDF_OK;                         \
>        }                                              \
>      *dest = dividend/divisor;                        \
If there isn't destination because it's NULL, why you store on it the
result? This line shouldn't be in the else clause?
>    } while(0)

I hope that help you.

Happy hacking:-)


Reply via email to