[mythtv] cvs 02-12 works, cvs 02-23 doesn't

[Bruce Markey]

Isaac Richards wrote:
> On Sunday 23 February 2003 12:59 pm, Eli Criffield wrote:
> 
>>line 346 was already 'audio_buffer_unused = 0;' but it was conditional on
>>line 345 'if(audio_buffer_unused < 0)', so the first thing i tried was
>>updating cvs and recompiling, same problem, except livetv worked fine (but
>>that mite have been because i uncommented out ServerMode=Master in
>>backend_settings.txt that i didn't before)
>>
>>I then added  audio_buffer_unused = 0; at line 347, thus making it
>>unconditional. And the same thing happens on playback and livetv now has a
>>bit of a ship to it about every second.
>>
>>I am getting "Audio buffer overflow, audio data lost!" in playback, but
>>those start about the same time it decides to play in fast mode and the
>>audio comes un-synced.
> 
> 
> Ok, can you remove any mods you made to the source, update to current CVS, and 
> try with both 'jitter reduction' and 'aggressive soundcard buffereing' 
> unchecked under the first screen of the Playback settings?
> 
> If that doesn't work, I'd need to know more exactly when it broke for you.  A 
> range of the 12th to the 21st is a little big =)

One other thing to test that may be related. In
MC/libs/libmythtv/NuppelVideoPlayer.cpp there are two
reference to a delay value of 40000. Change these to
200000 and uncomment the debug message. This is the
short pauses at startup while audio and video to synch
up. If the debug messages stop printing after the first
second or two but the problem persists, then this is not
the source of the problem.

--  bjm
-------------- next part --------------
Index: MC/libs/libmythtv/NuppelVideoPlayer.cpp
===================================================================
RCS file: /var/lib/cvs/MC/libs/libmythtv/NuppelVideoPlayer.cpp,v
retrieving revision 1.138
diff -u -r1.138 NuppelVideoPlayer.cpp
--- MC/libs/libmythtv/NuppelVideoPlayer.cpp	23 Feb 2003 23:13:26 -0000	1.138
+++ MC/libs/libmythtv/NuppelVideoPlayer.cpp	24 Feb 2003 00:13:14 -0000
@@ -1681,10 +1681,10 @@
         {
             /* If delay is sometwhat more than a frame or < 0ms, 
                we clip it to these amounts and reset nexttrigger */
-            if ( delay > 40000 )
+            if ( delay > 200000 )
             {
-                // cerr << "Delaying to next trigger: " << delay << endl;
-                delay = 40000;
+                cerr << "Delaying to next trigger: " << delay << endl;
+                delay = 200000;
                 usleep(delay);
 
                 gettimeofday(&nexttrigger, NULL);