[FFmpeg-devel] [PATCH] AAC Encoder: clipping avoidance

Claudio Freire klaussfreire at gmail.com
Sat Jul 18 01:19:23 CEST 2015


On Fri, Jul 17, 2015 at 7:09 PM, Rostislav Pehlivanov
<atomnuker at gmail.com> wrote:
>>attachment.bin:357: trailing whitespace.
>>attachment.bin:436: trailing whitespace.
>>attachment.bin:454: trailing whitespace.

Oops

> You should probably configure your editor to strip that off automatically.

I don't like it to be done automatically since it could (and in
general would) modify parts of the code I didn't touch should they
contain whitespace.

I prefer doing it manually. I guess those slipped past me. I thought I
had taken care of patchcheck issues already.

>
>>sce->sf_idx[win*16+swb],
> Why the change from (win+w)*2? Shouldn't you remove the entire loop around
> w because now nothing depends on it?

AFAIR, it's not possible to select different scalefactors for the
different windows in the window group, and a lot of code assumes that
only the first window has the proper scalefactor set.

I'm not sure, but I'd rather err on the safe side. Indexing on sf_idx
should always index the first group to avoid inconsistencies.

The loop however isn't rendered pointless by that change, there's
still the scoefs pointer being advanced by w (and that's the whole
point of the loop in fact).


More information about the ffmpeg-devel mailing list