[gstreamer-bugs] [Bug 341563] [faad] fails to decode middle of file, should just skip occasional stream errors

[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 #65389|none                        |committed
               Flag|                            |
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
            Summary|faad plugin is failed decode|[faad] fails to decode
                   |some file.                  |middle of file, should just
                   |                            |skip occasional stream
                   |                            |errors
   Target Milestone|HEAD                        |0.10.4




------- Comment #5 from Tim-Philipp Müller  2006-05-13 18:46 UTC -------
I also said you should reset the error counter whenever a decode operation was
successful :)  Committed to CVS, including that counter reset:

 2006-05-13  Tim-Philipp Müller  <tim at centricular dot net>

       Patch by: Young-Ho Cha  <ganadist chollian net>

       * ext/faad/gstfaad.c: (gst_faad_init), (gst_faad_chain),
       (gst_faad_change_state):
       * ext/faad/gstfaad.h:
         If we encounter a decoding error, don't error out immediately,
         but try to resync (or see if we have better luck with the next
         buffer in case of framed input). Only error out after five
         consecutive errors. Fixes #341563.

Thanks for the patch!


-- 
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.