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 172242 - gstspeed doesn't handle seek events correct
gstspeed doesn't handle seek events correct
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
git master
Other Linux
: Normal normal
: 0.8.9
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-03-31 12:56 UTC by Wouter Paesen
Modified: 2005-03-31 22:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix for the seek problem (3.79 KB, patch)
2005-03-31 12:58 UTC, Wouter Paesen
none Details | Review
adjust timestamps upstream AND downstream (4.21 KB, patch)
2005-03-31 16:19 UTC, Wouter Paesen
needs-work Details | Review
adjust timestamps upstream AND downstream (corrected) (5.20 KB, patch)
2005-03-31 19:44 UTC, Wouter Paesen
none Details | Review

Description Wouter Paesen 2005-03-31 12:56:48 UTC
upstream seeks will be forwarded by the speed plugin without adjustment.  This
is only correct when the speed is 1.
Comment 1 Wouter Paesen 2005-03-31 12:58:24 UTC
Created attachment 39502 [details] [review]
fix for the seek problem

This adds a method for handling upstream events. 
It also adds a getcaps function for correct capsnego
Comment 2 Wouter Paesen 2005-03-31 16:19:07 UTC
Created attachment 39516 [details] [review]
adjust timestamps upstream AND downstream

In addition to implementing upstream seek handling, this patch also adds
downstream timestamp adjustment
Comment 3 Ronald Bultje 2005-03-31 18:30:54 UTC
Hi, apart from creating a new event in the event handler, you also have to
create a new event accordingly in the chain function.
Comment 4 Wouter Paesen 2005-03-31 19:44:09 UTC
Created attachment 39522 [details] [review]
adjust timestamps upstream AND downstream (corrected)

Creation of new event downstream is handled now.
Comment 5 Ronald Bultje 2005-03-31 22:24:01 UTC
Thanks, applied.