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

sprezzatura sprezzatura
Sat Nov 27 02:48:59 CET 2010


Hello,

> Just a little remark, if you would not have submitted a patch that successfully
> decodes the sample you've uploaded, the original waveform would have been nice
> to have, too.

Ok, I will do so next time. (Anyway you can always get back the original wave
using the Reference Software decoder... ;))

> See my review below:
> 
> ...
> > - int const_block; ///< if true, this is a constant value block
> > - int32_t const_val; ///< the sample value of a constant block
> ?^^^^^^^^^
> 
> Removing const_val is unrelated, see comments below.
> ...
> 
> > - int store_prev_samples;///< if true, carryover samples have to
> be stored
> ?^^^^^^^^^^^^^^^^^^
> 
> Unrelated, more comments below.
> ...

Thanks for the review. I wanted to avoid duplicating the variables
unnecessarily because it involves adding many lines of extra code.
(That, and store_prev_samples screws up the vertical alignment. :D)

In the future it might make more sense to duplicate the whole
ALSBlockData structure per channel, since I think it would make
the code shorter.

But for now, I attached a patch without those variables removed,
and duplicated for all channels.

> Nice work except the things I've mentioned.

Thanks!

Regards,

Sprezz
-------------- next part --------------
A non-text attachment was scrubbed...
Name: als-mcc-bug-block-parameters_r2.diff
Type: application/octet-stream
Size: 13283 bytes
Desc: Attachment: als-mcc-bug-block-parameters_r2.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101127/8593ec07/attachment.obj>



More information about the ffmpeg-devel mailing list