[Bug 641204] dvbsrc: Faster tuning logic: Use poll instead of artificial usleep
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Mon May 26 21:03:13 PDT 2014
https://bugzilla.gnome.org/show_bug.cgi?id=641204
GStreamer | gst-plugins-bad | git
Reynaldo H. Verdejo Pinochet <reynaldo> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #179848|0 |1
is obsolete| |
Attachment #179950|0 |1
is obsolete| |
--- Comment #25 from Reynaldo H. Verdejo Pinochet <reynaldo at opendot.cl> 2014-05-27 01:49:44 UTC ---
Created an attachment (id=277252)
View: https://bugzilla.gnome.org/attachment.cgi?id=277252
Review: https://bugzilla.gnome.org/review?bug=641204&attachment=277252
Tuning: s/sleep()/poll + configurable timeouts + extras
Drops sleep logic in favor of configurable timeouts/polling
plus some extra refactors and improvements.
Most of the "extras" made sense on a single change, some
I might had handled separated (like the new signal bits)
but the thing grow organically as I attempted to match
Fabrizio's patch breakdown scheme, my bad.
Patch should apply cleanly over today's master, and has been
tested to work (with ATSC mostly).
Would love some hints/corrections specially on the polling /
timeout bits. I'm unsure as to whether moving the frontend's
poll vars to the GstDvbSrc struct would be desirable, maybe
you would like me to provide the user with a way to cancel
the main gst_poll_wait() ? I thought about pushing that idea
forward but would like to have your take on it first.
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
More information about the gstreamer-bugs
mailing list