[FFmpeg-devel] [PATCH 1/3] AviSynth demuxer rewrite.

[Michael Niedermayer]

On Tue, Feb 26, 2013 at 06:59:07PM -0500, Stephen Hutchinson wrote:
> From: d s <avxsynth.testing at gmail.com>
> 
> Dynamically loads the library itself, rather than through VfW.
> Supports AvxSynth on Linux and OS X.
> 
> Supports the new colorspaces added in AviSynth 2.6 when used
> with AviSynth 2.6.
[...]

> +#undef malloc
> +#undef free
> +#undef printf

unneeded


[...]

> +    av_freep(&avs_library);

> +    avs_library = NULL;

redudant


[...]

> +    pkt->size = ((avs->vi->width * avs->vi->height) * bits) / 8;

can the multiplication overflow before the /8 ?



also the avx/avisynth code in ffmpeg will need a maintainer
are you or someone else from the avxsynth team interrested?
If so please add yourself to the MAINTAINERS file

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20130227/de2268f6/attachment.asc>