[Ffmpeg-cvslog] r8427 - trunk/libavformat/utils.c

The Wanderer inverseparadox
Sat Mar 17 03:46:01 CET 2007


michael wrote:

> Author: michael
> Date: Sat Mar 17 03:03:59 2007
> New Revision: 8427
> 
> Modified:
>    trunk/libavformat/utils.c
> 
> Log:
> simplify

> +        if (pkt->dts == AV_NOPTS_VALUE)
> +            pkt->dts = st->last_IP_pts;
> +        if (pkt->dts == AV_NOPTS_VALUE)
> +            pkt->dts = st->cur_dts;

Am I blind, or are these two if()s testing exactly the same condition
and then setting exactly the same variable to two different values? That
can't be right...

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

Secrecy is the beginning of tyranny.




More information about the ffmpeg-cvslog mailing list