<div dir="ltr">Hi<div>Thank you for reviewing my patch.</div><div><br></div><div>Please see my reply in blue below.</div><div><br>On Wed, Jul 29, 2015 at 5:01 PM, Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br>>    Hello,<br>><br>> Le 2015-07-28 21:55, Yupeng Chang a écrit :<br>>><br>>> at line 708, the orginal code is<br>>> if ( !date_Get( &p_sys->end_date ) )<br>>>     date_Set( &p_sys->end_date, p_speex_bit_block->i_dts );<br>>><br>>> This is not correct.<br>>> When end_date is 0, the first packet's PTS will be set to end_date,<br>>> after RTCP sync,<br>>> the packet's PTS/DTS will be much different than the first packet's<br>>> PTS/DTS,<br>>> if the end_date is not updated, decoder will always report error message<br>><br>><br>> I think this code was copied from the non-RTP version above. I think you are<br>> right that it will break if the first RTCP SR compound is received after the<br>> first RTP packet (or if NTP/RTP time correlation is modified later).<br>><br>> In my opinion, Speex over RTP should not be treated as a codec of its own<br>> anyway. RTP depayloading and RTCP processing belongs in the demuxer.<br>><br>>> After I remove if ( !date_Get( &p_sys->end_date ) ), speex RTP playing<br>>> back works perfect with VLC.<br>><br>><br>> That would probably work as a stop-gap solution. But then I guess you should<br>> remove the date_Increment() call. In fact, you should probably remove<br>> date_*() calls completely and just copy the i_pts value from input block to<br>> output block.<br><br><span style="background-color:rgb(255,255,255)"><font color="#0000ff">OK, I'll remove date_*() calls and make a new patch.</font></span><br><br><br>> Ideally, the RTP processing would be moved out of the codec plugin into the<br>> RTP plugin, though. Are you using live555?<br><br></div><div><font color="#0000ff">I haven't studied VLC source code, so I'm lack of some architecture knowledge of some plugins.</font></div><div><font color="#0000ff">I think I can refine my patch and give a quick fix on this obvious bug.</font></div><div><font color="#0000ff"><br></font></div><div><font color="#0000ff">I implemented my own RTSP server, and I use VLC as the standard client player to test my RTSP server.</font></div><div><font color="#0000ff">So, I don't use live555 library.</font></div><div><br></div><div><br>> --<br>> Rémi Denis-Courmont<br>> <a href="http://www.remlab.net/">http://www.remlab.net/</a><br>><br>> _______________________________________________<br>> vlc-devel mailing list<br>> To unsubscribe or modify your subscription options:<br>> <a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br><br></div></div>