[FFmpeg-devel] [PATCH 1/2] libavcodec: Implementation of 32 bit fixed point FFT

Michael Niedermayer michaelni at gmx.at
Thu Feb 7 17:48:10 CET 2013


Hi

On Wed, Feb 06, 2013 at 03:12:24PM +0000, Babic, Nedeljko wrote:
> Hello,
> 
> Has someone had a chance to review this patch?

[...]
> +int32_t w_tab_sr[MAX_FFT_SIZE/(4*16)] = {
> +    2147483648, 2147481121, 2147473542, 2147460908, 2147443222, 2147420483, 2147392690, 2147359845,

The first element is not representable in int32_t

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

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- 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/20130207/5fdb9ab1/attachment.asc>


More information about the ffmpeg-devel mailing list