[FFmpeg-cvslog] r25404 - trunk/libavformat/rtpdec.c

Ronald S. Bultje rsbultje
Fri Oct 8 22:25:29 CEST 2010


Hi,

On Fri, Oct 8, 2010 at 4:40 AM, mstorsjo <subversion at mplayerhq.hu> wrote:
> Author: mstorsjo
> Date: Fri Oct ?8 10:40:08 2010
> New Revision: 25404
>
> Log:
> rtpdec: Set prev_ret properly when parsing more data from mpegts RTP packets

I'm not a big fan of this, I'm quit sure most of these aren't strictly
needed. Can't prev_ret be a cache variable only for the one case where
prev_ret and the actual return value are different, as originally
intended, and the rest be handled regardless of its value? We didn't
need to store the last return value previously either, and it worked
fine...

Ronald



More information about the ffmpeg-cvslog mailing list