Vous (Pierre Marchand) avez écrit :
> I’m getting into your recent changes and had a couple of trivial issues I
> wan’t to know if I must fix it myself or wait for you finish the work.
>
> + podofo_config.h is not installed.
> + HAVE_FONTCONFIG is publicly defined in podofo_config.h but not prefixed
> with "PODOFO_".

Going further I see there are rather more changes than I thought at first 
sight. 
It’s still not clear for me if podofo_config.h is intended to be public or 
just used at build time, I did notice it was missing when building apps 
relying on podofo that complained PDF_INT*_TYPENAME were not naming types.

Regarding deprecation of pdf_long as a  ptrdiff_t, the reasons for it are 
quite clear but I don’t see exactly what’s the new status of pdf_long. for 
exemple, when I see it in use in PdfMemStream::GetCopy, I wonder how it will 
behave when the uint64_t will be interpreted as a size_t in the memcpy call.

I continue to read the new stuff…


-- 
Pierre Marchand

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

Reply via email to