[Gstreamer-bugs] [Bug 104185] Changed - ESDSINK does not sync

[bugzilla-daemon at widget.gnome.org]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=104185

Changed by ds at schleef.org.

--- shadow/104185	Thu Jan 23 08:21:55 2003
+++ shadow/104185.tmp.8407	Wed Jul 23 03:59:52 2003
@@ -1,13 +1,13 @@
 Bug#: 104185
 Product: GStreamer
-Version: cvs
+Version: HEAD CVS
 OS: Linux
 OS Details: Gentoo Linux
-Status: ASSIGNED   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: major
 Priority: High
 Component: gst-plugins
 AssignedTo: gstreamer-maint at bugzilla.gnome.org                            
 ReportedBy: mathewalexander at surewest.net               
 QAContact: gstreamer-maint at bugzilla.gnome.org
@@ -28,6 +28,25 @@
 
 ------- Additional Comments From thomas at urgent.rug.ac.be  2003-01-23 08:21 -------
 Yeah, I've noticed esd isn't working properly too yesterday.
 
 Would be nice to get this fixed, maybe someone should comment on what
 should be done to get it right.
+
+------- Additional Comments From ds at schleef.org  2003-07-23 03:59 -------
+esdsink can't sync properly because it doesn't have a clock.  Nobody
+bothered to add a clock to esdsink because the esd daemon doesn't
+report the latency from samples being written to the actual audio
+output.  Thus, any attempt at synchronization relies on heuristics and
+guesses.
+
+I recently added a clock to esdsink in HEAD, just because it's easy
+now (due to various other changes).  The results are uninteresting. 
+Instead of being "way off", it's just "a bit off", with about the same
+level of annoyance.
+
+I have no intention of backporting this to 0.6.  Esd is simply not
+capable of synchronizing with video.  Thus, I'm marking the bug fixed.
+ If further discussion is warranted, please reopen.
+
+(Note that there are semi-related patches to esdsink that I do plan to
+backport, since they affect bugs that are actually solvable.)