[Bug 737339] gifdec: new element

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=737339
  GStreamer | gst-plugins-bad | unspecified

--- Comment #4 from Vineeth <vineeth.tm at samsung.com> 2014-09-25 11:12:18 UTC ---
Created an attachment (id=287060)
 View: https://bugzilla.gnome.org/attachment.cgi?id=287060
 Review: https://bugzilla.gnome.org/review?bug=737339&attachment=287060

Adding a new gifdec element

Hi Sebastian,

I am attaching the patch now :)
The same can be tested with the following launch command
gst-launch-1.0 -v filesrc location=../1.gif ! image/gif,framerate=5/1 ! gifdec
! videoconvert ! ximagesink

i was referring to libav to check for gif decoders and there this
implementation of lzw was being used. Hence i followed the same.
Probably the next step would be to use giflib instead of our own lzw
implementation. Since i had to start somewhere, i did the same by using already
available code.


Hi Tim,

There is a gif decoder in libav. But it is disabled.
Even when i enable it, it just shows the first image similar to gdkpixbufdec

I had read comments in forums about you already trying with gdkpixbufdec,
hence i implemented the same by decoding one frame at a time and displaying the
same before decoding the next frame.
I parse the gif file, until the end of one image is found, and then post finish
frame, which displays the frame. and then continue parsing until end of next
image is found.

It did seem to work OK. Please review the changes.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.