Changeset 369 for OCO/trunk/drv16/wavestrm.cpp

Timestamp:

Sep 3, 2008, 4:08:49 PM (17 years ago)

Author:

rjerant

Message:

Fixed ticket:18 - streaming problem with video playback

File:

• OCO/trunk/drv16/wavestrm.cpp (modified) (3 diffs)

OCO/trunk/drv16/wavestrm.cpp

@@ -299 +299 @@
     case STREAM_WRITE:
     {
-        if(pahw->GetPosition(StreamId, &_configinfo, &ulCurBytesProcessed) == FALSE ||
+        if(pahw->GetPosition(StreamId, &_configinfo, &ulCurBytesProcessed) == FALSE || 
            ulCurBytesProcessed == 0) 
         {//shouldn't happen
@@ -318 +318 @@
 
             ulStreamPos  = pahw->ConvertPositionInvert(ulCurBytesProcessed, &_configinfo);
-             //round to sample boundary
+            //round to sample boundary
             ulStreamPos &= ~(_configinfo.ulSampleSize - 1);
 
 #ifdef DEBUG
-              dprintf(("WARNING: Process: Current pos %ld pos %ld incr %d", ulCurBytesProcessed, _ulBytesProcessed, bytesinc));
-              dprintf(("WARNING: hw pos %lx, streampos %lx", ulCurBytesProcessed, ulStreamPos));
+            dprintf(("WARNING: Process: Current pos %ld pos %ld incr %d", ulCurBytesProcessed, _ulBytesProcessed, bytesinc));
+            dprintf(("WARNING: hw pos %lx, streampos %lx", ulCurBytesProcessed, ulStreamPos));
 #endif /* DEBUG */
         }
@@ -330 +330 @@
         while(bytesinc) 
         {
-            if(qhDone.IsElements()) {  // if there are buffers that have been
-                                       // completly written to the audio buffer
+          if(qhDone.IsElements()) {  // if there are buffers that have been completly written to the audio buffer
                                        // check the first one on the done queue
-                                       // if it's data has been consumed by
-                                       // the hardware return it
+                                       // if it's data has been consumed by the hardware return it.
+                                                         // Bug #18 Sometimes MMPM gives us really small buffers. If we 
+                                                         // wait for the buffer to be completly consumed and there is a tiny buffer
+                                                         // on the queue after this one, the sound part could underrun before we get a 
+                                                         // chance to refill it. To fix this we will return the buffer if it will be  
+                                                         // consumed before the next call to WAVESTREAM:Process(). 
+                                                         // rjj Fix for Bug #18 28082008
                 ptemp = (PSTREAMBUFFER)qhDone.Head();
                 ptemp->ulDonepos += bytesinc;
                 bytesinc          = 0;
-                if(ptemp->ulDonepos >= ptemp->ulBuffsz) 
+                                  if ((ptemp->ulDonepos + _configinfo.ulBytesPerIRQ) >= ptemp->ulBuffsz)
                 {
                     //calc position in next buffer
                     bytesinc = ptemp->ulDonepos - ptemp->ulBuffsz;
-//                    dprintf2(("Process: Return buffer %lx size %d", ptemp->pBuffptr, ptemp->ulBuffsz));
+//                  dprintf2(("Process: Return buffer %lx size %d", ptemp->pBuffptr, ptemp->ulBuffsz));
                     ReturnBuffer();
-                        }
-            }
+                          }
+          }
             else        break; //shouldn't happen
         }