[poppler] more patches to the qt4 interface
bradh at frogmouth.net
Sun Dec 11 11:55:21 PST 2005
On Monday 12 December 2005 04:31 am, Stefan Kebekus wrote:
> attached you'll find a patch (for the qt4-directory, ass usual) with the
> following changes:
> * fixed a problem in the textList() method, where coordinated were wrongly
That bit it is OK.
> * added more documentation
That is OK, except I don't see why you are using \brief. Normally that means
something is misconfigured in the Doxygen setup. Can you explain this bit?
> * ported the pageTransition class and the associated methods from the
The API mostly looks OK, but I'm not so keen on the parsing code in
poppler-document.cc. Can we push this down into the poppler common code?
Otherwise qt3 and qt4 will diverge in their parsing, and glib will get yet
another version. All this hairy "get this subdictionary from that object"
crud is fragile to specification interpretation variances, so we really want
it in once place.
Also, what about some automated tests for all this new API?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/poppler/attachments/20051212/b4ba02bb/attachment.pgp
More information about the poppler