[FFmpeg-devel] [PATCH] lavf remove duplicated check in has_duration

Nicolas George nicolas.george at normalesup.org
Sat Apr 28 14:56:45 CEST 2012


Le decadi 10 floréal, an CCXX, Jean First a écrit :
> this hunk was merged in 8b97ae64 and cbf767a8 although the check was there
> a few lines above since cdced09e

Are you sure?
> 
>          if (st->duration != AV_NOPTS_VALUE)
>              return 1;
>      }
> -    if (ic->duration != AV_NOPTS_VALUE)
> -        return 1;

The way I read it, the test a few lines above looks at "st", i.e. the
stream, while the test you want to remove looks at "ic", i.e. the whole
file.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120428/53ddb6d2/attachment.asc>


More information about the ffmpeg-devel mailing list