[FFmpeg-devel] [PATCH] MPEG-4 ALS decoder MCC decoding bug (non-lossless output) (Fixes issue 2387)

sprezzatura sprezzatura
Sat Nov 27 16:06:45 CET 2010


Hi,

> > But for now, I attached a patch without those variables removed,
> > and duplicated for all channels.
> 
> Review below:
> 
> ...
> This is not what I wanted to say with my last remark. Storing the constant value
> in ->raw_buffer like you did in the first place is ok!
> 
> > + int *store_prev_samples; ///< contains store_prev_samples flags
> for all channels
> 
> Storing these per channel definitely removes a bug and complies to this patch,
> so it is ok although I've no file in my test suite nor does your uploaded file
> suffer from this buggy field. Ok.
> 
> All other changes ok.
> 
> Getting close!

Sorry, I don't think I follow. Do you want me to store the const value in
->raw_buffer like in the first patch, and store store_prev_samples per channel
like in the second patch?

Regards,

Sprezz



More information about the ffmpeg-devel mailing list