On Fri, Nov 11, 2011 at 5:28 AM, Martin Storsjö <[email protected]> wrote: > On Thu, 10 Nov 2011, Luca Barbato wrote: > >> On 11/9/11 3:28 PM, John Brooks wrote: >>> >>> RTCP timestamps are only necessary to synchronize time between >>> multiple streams. For a single stream, the RTP packet timestamp >>> provides more reliable timing. As a result, single-stream RTP >>> sessions should now have accurate and monotonic PTS. >>> --- >> >> The patch looks fine, I'm traveling so I can't check it properly, Martin >> if it is ok for you then it is for me as well =) > > Applied the cleanup part of it, will apply the actual functional change > later. Do you think it's ok to apply it now (given that it makes timestamps > wrap around within 13 hours for mpeg codecs, wrapping around in a way that > an app using libavformat can't easily compensate for), or should we wait for > the fix for unwrapping plain RTP timestamps first?
I'll write a fix for timestamp wrapping early next week; it's probably best to wait on that. I do have a case where streams might go longer than 13 hours :) - John _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
