After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 504018 - flacdec emits newsegment before being able to respond to seeks properly
flacdec emits newsegment before being able to respond to seeks properly
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-12-17 10:13 UTC by Peteris Krisjanis
Modified: 2007-12-18 17:40 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Description Peteris Krisjanis 2007-12-17 10:13:07 UTC 
Please describe the problem:
This bug is following https://bugs.launchpad.net/jokosher/+bug/174474 and http://bugzilla.gnome.org/show_bug.cgi?id=502187. Bilboed comment: "flacdec emits newsegment before being able to answer properly to seeks, which makes it unusable with gnonlin".

Following Jokosher bug, now it doesn't freeze, but it ignore gaps between pieces.

Steps to reproduce:
Follow steps in Jokosher bug. Result is different (Jokosher doesn't freeze) but still buggy.

Actual results:
Jokosher ignores gaps between pieces in Instrument track. 

Expected results:
Jokosher makes silence when there is a gap between pieces.

Does this happen every time?
Yes.

Other information:
Comment 1 Edward Hervey 2007-12-17 10:14:04 UTC 
confirmed with gst-media-test gnltest
Comment 2 Wim Taymans 2007-12-18 17:40:47 UTC 
This makes jokosher work for me:

        * ext/flac/gstflacdec.c: (gst_flac_dec_reset_decoders),
        (gst_flac_dec_update_metadata), (gst_flac_dec_metadata_callback),
        (gst_flac_dec_write):
        * ext/flac/gstflacdec.h:
        Remove some unused vars.
        Do more cleanup of leftover events and tags.
        Output tags after the segment event. Fixes #504018.