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 761906 - testclock: add crank method
testclock: add crank method
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: Normal enhancement
: 1.7.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 761905
Blocks: 761932
 
 
Reported: 2016-02-12 10:05 UTC by Håvard Graff (hgr)
Modified: 2016-02-12 17:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
test and patch (7.16 KB, patch)
2016-02-12 10:05 UTC, Håvard Graff (hgr)
none Details | Review
updated patch (7.15 KB, patch)
2016-02-12 12:18 UTC, Håvard Graff (hgr)
none Details | Review
updated patch (7.15 KB, patch)
2016-02-12 14:06 UTC, Håvard Graff (hgr)
committed Details | Review

Description Håvard Graff (hgr) 2016-02-12 10:05:26 UTC
Created attachment 320944 [details] [review]
test and patch

The "crank" can also be useful outside of GstHarness, doing the 3 steps needed. (wait for clock-id-wait, advance the clock to the time waited for, and release the wait)

Also have GstHarness use this new method internally!
Comment 1 Håvard Graff (hgr) 2016-02-12 12:18:36 UTC
Created attachment 320967 [details] [review]
updated patch
Comment 2 Håvard Graff (hgr) 2016-02-12 14:06:01 UTC
Created attachment 320977 [details] [review]
updated patch
Comment 3 Tim-Philipp Müller 2016-02-12 17:13:03 UTC
commit 668b3215b80625aa6f923eaa645b330925fc5b8f
Author: Havard Graff <havard.graff@gmail.com>
Date:   Thu Jan 14 21:54:42 2016 +0100

    testclock: add crank method
    
    And use it inside GstHarness
    
    API: gst_test_clock_crank()
    
    https://bugzilla.gnome.org/show_bug.cgi?id=761906
Comment 4 Tim-Philipp Müller 2016-02-12 17:14:21 UTC
Comment on attachment 320977 [details] [review]
updated patch

Committed with a minor change, removed an empty line in the gtk-doc chunk which shouldn't be there (between func name and func argument).