[FFmpeg-devel] [PATCH] movie video source
Michael Niedermayer
michaelni
Sun Jan 30 16:15:10 CET 2011
On Sun, Jan 30, 2011 at 12:32:25AM +0100, Stefano Sabatini wrote:
[...]
> + if (movie->seek_point_d > (INT64_MAX-0.5) / 1000000) {
> + av_log(ctx, AV_LOG_ERROR, "Value for seek point is too big\n");
> + return AVERROR(EINVAL);
> + }
i think you can set the max in the AVOptions table correctly
except that lgtm if tested
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110130/5158f086/attachment.pgp>
More information about the ffmpeg-devel
mailing list