[poppler] [PATCH] Preliminary movie support
Hugo Mercier
hmercier31 at gmail.com
Sat Mar 1 09:51:46 PST 2008
>
> Not commenging in glib code as we have better people to that.
>
> Minor comments:
> * Use poppler own types in poppler core code (yeah i know it sucks)
> * Use poppler's Gulong instead of long in poppler core core
> * Use poppler's gFalse and gTrue in poppler core core
> * Use GBool on Movie.h
> * You killed "Sound" from the if on createAnnot, is that a mistake?
>
> Important things:
> * The Movie ownernship is not clear to me, AnnotMovie creates one but
> never
> deletes it, that makes an instant leak. Same for LinkRendition. I think
> your
> idea is that people needs to call getMovie and then take care of deleting
> themselves. That's unaccetable because not everyone might want to care
> about
> Annots and because if there are two calls to getMovie it creates a "who
> has
> to delete dilemma". Probably the easiest solution is making the getMovie
> functions return a copy of movie the caller has to care to delete.
>
> Critical things:
> * Movie.h is GPLv3+ which makes it impossible for us to use it, it needs
> to
> be GPLv2+
>
>
Hi,
Here is another patch (against the last git commit) that fixes these issues.
Ownership of members has been clarified.
I only send a patch that adds movie support to the "core" poppler lib. Movie
access in the glib part has still to be rewritten to conform to recent
changes proposed by Iñigo (PopplerAnnotMapping).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freedesktop.org/archives/poppler/attachments/20080301/2fff0394/attachment-0001.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: poppler_core_movie_support3.patch
Type: text/x-patch
Size: 36047 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/poppler/attachments/20080301/2fff0394/attachment-0001.bin
More information about the poppler
mailing list