[gstreamer-bugs] [Bug 382592] Multiple wait()ers break periodic clock ids

[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=382592

  GStreamer | gstreamer (core) | Ver: HEAD CVS


Wim Taymans changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #77726|none                        |committed
               Flag|                            |
  Attachment #77727|none                        |committed
               Flag|                            |
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




------- Comment #4 from Wim Taymans  2006-12-07 10:52 UTC -------
        Patch by: Ren�� Stadler <mail at renestadler dot de>

        * gst/gstclock.c: (gst_clock_id_wait):
        Make period ids add the interval to the origial requested time instead
        of the possibly updated time which can be wrong when there are multiple
        waiters for the same id. Fixes #382592.

        * gst/gstsystemclock.c: (gst_system_clock_async_thread),
        (gst_system_clock_id_wait_jitter_unlocked),
        (gst_system_clock_id_wait_jitter):
        Fix restart in the async notify thread when an async entry is added to
        the front of the list. Fixes #381492.

        * tests/check/gst/gstsystemclock.c: (store_callback),
        (notify_callback), (GST_START_TEST), (gst_systemclock_suite):
        Added test for multiple async waits.
        Added test for async wait order.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email