[FFmpeg-devel] [PATCH 3/3] [GSoC] [AAC] aacdec: Use defined macros for constants

Claudio Freire klaussfreire at gmail.com
Mon Apr 13 05:05:57 CEST 2015


On Mon, Apr 13, 2015 at 12:02 AM, Claudio Freire <klaussfreire at gmail.com> wrote:
> Looks good.
>
> Though it does depend on the first patch of the series.
>
>
> On Sun, Apr 12, 2015 at 1:50 AM, Rostislav Pehlivanov
> <atomnuker at gmail.com> wrote:
>> This commit changes the decoder to use the macro definitions for scalefactors and noise energy values reading. This was done to reduce confusion when looking at the encoder and decoder and possibly as a first step in unifying some of the code in the two files.

Another thing I'd like to see in this patch set is a switch to enable
setting NOISE_BT, default off.

This is still a POC and should not be enabled unless specifically asked for.


More information about the ffmpeg-devel mailing list