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 739965 - validate: Assumes first buffer should have a running time of 0
validate: Assumes first buffer should have a running time of 0
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-devtools
git master
Other Linux
: Normal normal
: 1.5.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-11-11 14:24 UTC by Sebastian Dröge (slomo)
Modified: 2015-06-24 14:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
validate: Do not check if first buffer running time is 0 (14.82 KB, patch)
2014-11-11 20:01 UTC, Thibault Saunier
committed Details | Review

Description Sebastian Dröge (slomo) 2014-11-11 14:24:35 UTC
In gst_validate_pad_monitor_check_first_buffer() there is a check that asserts that the first buffer's running time should always be 0. This is not correct in at least two cases:
- live pipelines
- container files where e.g. the audio stream starts a bit later than the video stream, thus the first audio running time is > 0
Comment 1 Thibault Saunier 2014-11-11 19:33:33 UTC
OK, I was wondering also, we should just remove that check then or can you think of a way to properly check that?
Comment 2 Sebastian Dröge (slomo) 2014-11-11 19:45:15 UTC
What do you want to check for? :) I can't think of anything generic that could be checked there
Comment 3 Thibault Saunier 2014-11-11 20:01:12 UTC
Created attachment 290431 [details] [review]
validate: Do not check if first buffer running time is 0

It can perfectly not be 0, so it makes no sense to check that.
Comment 4 Sebastian Dröge (slomo) 2014-11-12 08:58:07 UTC
Comment on attachment 290431 [details] [review]
validate: Do not check if first buffer running time is 0

Looks good... but that means you're not committed to a stable API for gst-validate (yet)?
Comment 5 Thibault Saunier 2014-11-12 09:08:57 UTC
There are very few APIs that are declared stable, they are the ones that have been documented, the rest is still "moving". That is what I explained in the release notes.
Comment 6 Thibault Saunier 2014-11-19 17:01:52 UTC
Attachment 290431 [details] pushed as 7f8c0c2 - validate: Do not check if first buffer running time is 0