[poppler] glib movie support.

[Carlos Garcia Campos]

Excerpts from Sam Kaplan's message of mar nov 10 19:02:10 +0100 2009:
> On Tue, 2009-11-10 at 18:18 +0100, Carlos Garcia Campos wrote:
> > Excerpts from Sam Kaplan's message of lun nov 09 02:01:34 +0100 2009:
> > > Hi,
> > 
> > Hi Sam, 
> > 
> > > I found a couple of problems in the patch that I sent yesterday.  Please
> > > find attached an updated patch (patch-movie).
> > 
> > Thank you very much for the patch. Hugo Mercier already made a patch
> > for this some time ago, see:
> > 
> > http://lists.freedesktop.org/archives/poppler/2008-August/004012.html
> > http://lists.freedesktop.org/archives/poppler/2008-September/004064.html
> 
> Thank-you very much for the links!
> > 
> > I haven't had time to look at your patch in detail yet, so I don't
> > know whether it's the same thing that Hugo wrote, but it's probably a
> > good time to take this up again.
> > 
> 
> It looks like there is overlap.  For example, we both wrote
> poppler-movie.h/.c files.  Hugo's poppler-movie.h/.c files are more
> complete then mine.  When I get time (which is limited due to my PhD
> theses work), I'll work to use Hugo's poppler-movie.h/.c files in-place
> of mine.
> 
> I generated my pdf example using the LaTeX movie15 package.  This
> package seems to generate movies that are accessed by the
> 'Annot::typeScreen'.  Hugo's patch uses the 'Annot::typeMovie'.  This
> led me to add AnnotScreen code, while Hugo's patch adds AnnotMovie code.
> So, from this point of view the patches compliment each other (unless I
> made a mistake).
> 
> Maybe we can make these into three patches.  1) Hugo's
> poppler-movie.h/.c files. 2) Hugo's AnnotMovie code. 3) Sam's
> AnnotScreen code.

I've finally found some time to look at glib movie support. I've reworked
the whole thing a little bit, and pushed a patch mostly based on your
patches and Hugo's. Thank you both!. 

> Thanks,
> 
> Sam
> 
> > Hugo?
> > 
> > > Thanks,
> > > 
> > > Sam
> > > 
> > > On Sat, 2009-11-07 at 18:31 -0700, Sam Kaplan wrote:
> > > > Hi,
> > > > 
> > > > I've written a patch to add movie support for glib (which I think was
> > > > missing from the glib bindings).  With this patch, you can now save a
> > > > movie to a file using the 'glib/demo/poppler-glib-demo' program.
> > > > 
> > > > I've attached the patch, and an example file with a movie.  To save the
> > > > movie to a file, do:
> > > > 
> > > > glib/demo/poppler-glib-demo test.pdf
> > > > (click on Annots->get Annots)
> > > > (click on the second entry in the table)
> > > > you should see a file dialog so that you can save the movie to disk.
> > > > 
> > > > This is my first try at writing c code for glib, and my first time
> > > > making a patch.  Feedback welcome.
> > > > 
> > > > Sam
> > > > 
> > > > _______________________________________________
> > > > poppler mailing list
> > > > poppler at lists.freedesktop.org
> > > > http://lists.freedesktop.org/mailman/listinfo/poppler
> > _______________________________________________
> > poppler mailing list
> > poppler at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/poppler
-- 
Carlos Garcia Campos
PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20100315/f789412a/attachment.pgp>