Excellent!
Actually, what I was hoping for was that the Poppler guys might define a version macro somewhere in their API .h files. But this is good, too.
bob
Peter Moulder wrote:
On Sun, Jun 15, 2008 at 11:32:05PM -0500, Bob Jamison wrote:
No, it's not an autoconf option, but a CFLAGS setting. Either:
./configure CFLAGS="-DPOPPLER_NEW_GFXFONT"
or
env CFLAGS="-DPOPPLER_NEW_GFXFONT" ./configure
Should do the job.
Actually, that should be CPPFLAGS, not CFLAGS: `-D...' and `-I...' flags are for the preprocessor, not the C compiler.
In any case, I've now added a test to configure.ac so that the user needn't manually specify this flag, given that this `temporary hack' is still in place after a month.
I've also added a TODO comment to the C++ code, pointing out what looks like a memory leak: without looking at the documentation or implementation of getNextChar, I suppose that u is getting set to new heap-allocated memory that we're supposed to free somehow, whereas we aren't freeing it.
pjrm.
Now, this is just a temporary hack that I made until Miklos has an opportunity to look at it. He already has an idea for a much better fix, where this flag will no longer be needed.
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK& win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel