[gstreamer-bugs] [Bug 300682] gst-player and Totem (gstreamer) fails to play mpgtx -j joined files to the end

[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=300682
 GStreamer | gstreamer (core) | Ver: HEAD CVS


Luca Ognibene changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
            Version|0.8.9                       |HEAD CVS




------- Comment #4 from Luca Ognibene  2006-02-25 11:55 UTC -------
Thanks for the file! 
I can replicate the bug with current cvs and also in mpeg2dec (the program) so
it seems an upstream issue. Using ffdec_mpeg2video instead mpeg2dec make it
works fine.
I've only seen a problem:
 - in gstmpeg2dec.c mpeg2_parse returns STATE_INVALID and mpeg2dec creates a
warning and return GST_FLOW_ERROR.. 
I think this is wrong, it should:
or return GST_FLOW_ERROR and call GST_ELEMENT_ERROR
or return GST_FLOW_OK and call GST_OBJECT_WARNING 

The problem is that: ... ! mpegdemux ! mpeg2dec ! fakesink works fine (just
some warnings) but ... ! mpegdemux name=m m.video_00 ! queue ! mpeg2dec !
fakesink fails because queue fails if the ret value is different than
GST_FLOW_OK.

Changing "return GST_FLOW_ERROR" to "return GST_FLOW_OK" make this file works
fine. Should i commit? 


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