[gstreamer-bugs] [Bug 341563] faad plugin is failed decode some file.

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=341563
 GStreamer | gst-plugins-bad | Ver: 0.10.x


Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #65385|none                        |needs-work
               Flag|                            |




------- Comment #2 from Tim-Philipp Müller  2006-05-13 15:13 UTC -------
Thanks for tracking this down, here are some comments, as already discussed on
IRC:

 - the error counter should be moved into the GstFaad structure and
   reset to 0 whenever a decode operation was successful. In the
   setup like in the patch where the error counter is local to the
   chain function, it would never be triggered for framed data and
   hardly ever for unframed data.

 - the limit for the error counter doesn't really make sense to me
   in the context of how the GStreamer decoder works (I presume
   mplayer does the resyncing in a different way than we do), it
   should probably just be

      #define MAX_DECODE_ERRORS 5

   or something (some magic number).


Btw, it would be great if you could make the first 5 or 10 megabytes of the
file available somewhere for download (or the whole file if it's not too big)
so we can add it to our test suite.


-- 
Configure bugmail: http://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.