[FFmpeg-devel] [PATCH 1/3] avcodec/aacsbr: Split the env_facs table

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Fri Dec 11 17:14:55 CET 2015


On 19.11.2015 14:17, Michael Niedermayer wrote:
> From: Michael Niedermayer <michael at niedermayer.cc>
> 
> This also removes a #ifdef and special case for the fixed point case
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/aacsbr.c          |    6 +++---
>  libavcodec/aacsbr_fixed.c    |    6 +++---
>  libavcodec/aacsbr_template.c |   42 ++++++++----------------------------------
>  3 files changed, 14 insertions(+), 40 deletions(-)

This looks good to me, because it makes the code shorter and easier to understand.

Best regards,
Andreas



More information about the ffmpeg-devel mailing list