<br><br>On Monday, July 31, 2017, Francois Cartegnie <<a href="javascript:_e(%7B%7D,'cvml','fcvlcdev@free.fr');" target="_blank">fcvlcdev@free.fr</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 31/07/2017 à 16:08, Francois Cartegnie a écrit :<br>
<br>
>>              mtime_t i_deadline = mdate();<br>
>> -            if (i_return == AbstractStream::buffering_ongo<wbr>ing)<br>
>> +            if(i_return == AbstractStream::buffering_ongo<wbr>ing)<br>
>>                  i_deadline += (CLOCK_FREQ / 20);<br>
>> -            if (i_return == AbstractStream::buffering_full<wbr>)<br>
>> +            else if(i_return == AbstractStream::buffering_full<wbr>)<br>
>>                  i_deadline += (CLOCK_FREQ / 10);<br>
>>              else if(i_return == AbstractStream::buffering_end)<br>
>>                  i_deadline += (CLOCK_FREQ);<br>
><br>
> I don't remember if I wanted to add delays, but,<br>
><br>
<br>
Err, that wasn't even adding delays.<br>
<br>
So you're not fixing anything in the logic. Just adding else into syntax.<br>
<br></blockquote><div><br></div><div>The original code is:</div><div><pre style="word-wrap:break-word">            mtime_t i_deadline = mdate();
            if (i_return == AbstractStream::buffering_ongoing)
                i_deadline += (CLOCK_FREQ / 20);
            if (i_return == AbstractStream::buffering_full)
                i_deadline += (CLOCK_FREQ / 10);
            else if(i_return == AbstractStream::buffering_end)
                i_deadline += (CLOCK_FREQ);
            else /*if(i_return == AbstractStream::buffering_suspended)*/
                i_deadline += (CLOCK_FREQ / 4);
</pre></div><div><br></div><div>So when i_return is AbstractStream::buffering_ongoing, i_deadline = mdate() + (CLOCK_FREQ / 20) + (CLOCK_FREQ / 4). 300 milliseconds are too much while buffering_ongoing. Actually 50 milliseconds don't work well too, but it's another topic.</div><div>   </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Francois<br>
______________________________<wbr>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/l<wbr>istinfo/vlc-devel</a></blockquote>