Comment 6 for bug 487038

Revision history for this message
Michael Witten (mfwitten) wrote :

From what I can tell, that patch is horrendously incorrect; it will allow inkscape to build, but then inkscape will only run as long as you never execute certain color space code in poppler (who knows what constraints the future will bring).

Specifically, inkscape's PdfParser class was apparently "derived" from poppler's Gfx class in the sense that the code was copied from Gfx.h a LONG time ago and then refactored; there is no formal derivation (that is, using Gfx as a base class). Ergo, casting (with a C-style cast, no less) the PdfParser (`this') to type Gfx* is non-portable over time and indeed makes no sense as of today.