[FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

Rostislav Pehlivanov atomnuker at gmail.com
Sun Jan 24 22:31:02 CET 2016


+1

I doubt anyone will complain or notice.

On 24 January 2016 at 16:33, Kieran Kunhya <kieran at kunhya.com> wrote:

> The internal encoder is superior to libvo-aacenc.
> ---
>  configure                 |   6 --
>  doc/encoders.texi         |  25 ------
>  doc/general.texi          |   8 --
>  libavcodec/Makefile       |   1 -
>  libavcodec/allcodecs.c    |   1 -
>  libavcodec/libvo-aacenc.c | 200
> ----------------------------------------------
>  libavcodec/version.h      |   2 +-
>  7 files changed, 1 insertion(+), 242 deletions(-)
>  delete mode 100644 libavcodec/libvo-aacenc.c
>
> diff --git a/configure b/configure
> index 35e3513..b940d2a 100755
> --- a/configure
> +++ b/configure
> @@ -255,7 +255,6 @@ External library support:
>    --enable-libutvideo      enable Ut Video encoding and decoding via
> libutvideo [no]
>    --enable-libv4l2         enable libv4l2/v4l-utils [no]
>    --enable-libvidstab      enable video stabilization using vid.stab [no]
> -  --enable-libvo-aacenc    enable AAC encoding via libvo-aacenc [no]
>    --enable-libvo-amrwbenc  enable AMR-WB encoding via libvo-amrwbenc [no]
>    --enable-libvorbis       enable Vorbis en/decoding via libvorbis,
>                             native implementation exists [no]
> @@ -1481,7 +1480,6 @@ EXTERNAL_LIBRARY_LIST="
>      libutvideo
>      libv4l2
>      libvidstab
> -    libvo_aacenc
>      libvo_amrwbenc
>      libvorbis
>      libvpx
> @@ -2639,8 +2637,6 @@ libspeex_encoder_deps="libspeex"
>  libspeex_encoder_select="audio_frame_queue"
>  libtheora_encoder_deps="libtheora"
>  libtwolame_encoder_deps="libtwolame"
> -libvo_aacenc_encoder_deps="libvo_aacenc"
> -libvo_aacenc_encoder_select="audio_frame_queue"
>  libvo_amrwbenc_encoder_deps="libvo_amrwbenc"
>  libvorbis_decoder_deps="libvorbis"
>  libvorbis_encoder_deps="libvorbis"
> @@ -4866,7 +4862,6 @@ die_license_disabled version3 gmp
>  die_license_disabled version3 libopencore_amrnb
>  die_license_disabled version3 libopencore_amrwb
>  die_license_disabled version3 libsmbclient
> -die_license_disabled version3 libvo_aacenc
>  die_license_disabled version3 libvo_amrwbenc
>
>  enabled version3 && { enabled gpl && enable gplv3 || enable lgplv3; }
> @@ -5488,7 +5483,6 @@ enabled libtwolame        && require libtwolame
> twolame.h twolame_init -ltwolame
>  enabled libutvideo        && require_cpp utvideo "stdint.h stdlib.h
> utvideo/utvideo.h utvideo/Codec.h" 'CCodec*' -lutvideo -lstdc++
>  enabled libv4l2           && require_pkg_config libv4l2 libv4l2.h
> v4l2_ioctl
>  enabled libvidstab        && require_pkg_config "vidstab >= 0.98"
> vid.stab/libvidstab.h vsMotionDetectInit
> -enabled libvo_aacenc      && require libvo_aacenc vo-aacenc/voAAC.h
> voGetAACEncAPI -lvo-aacenc
>  enabled libvo_amrwbenc    && require libvo_amrwbenc vo-amrwbenc/enc_if.h
> E_IF_init -lvo-amrwbenc
>  enabled libvorbis         && require libvorbis vorbis/vorbisenc.h
> vorbis_info_init -lvorbisenc -lvorbis -logg
>
> diff --git a/doc/encoders.texi b/doc/encoders.texi
> index 33699df..c485f90 100644
> --- a/doc/encoders.texi
> +++ b/doc/encoders.texi
> @@ -1075,31 +1075,6 @@ Set MPEG audio original flag when set to 1. The
> default value is 0
>
>  @end table
>
> - at anchor{libvo-aacenc}
> - at section libvo-aacenc
> -
> -VisualOn AAC encoder.
> -
> -Requires the presence of the libvo-aacenc headers and library during
> -configuration. You need to explicitly configure the build with
> - at code{--enable-libvo-aacenc --enable-version3}.
> -
> -This encoder is considered to be worse than the
> - at ref{aacenc,,native FFmpeg AAC encoder}, according to
> -multiple sources.
> -
> - at subsection Options
> -
> -The VisualOn AAC encoder only support encoding AAC-LC and up to 2
> -channels. It is also CBR-only.
> -
> - at table @option
> -
> - at item b
> -Set bit rate in bits/s.
> -
> - at end table
> -
>  @section libvo-amrwbenc
>
>  VisualOn Adaptive Multi-Rate Wideband encoder.
> diff --git a/doc/general.texi b/doc/general.texi
> index 06933ab..185c978 100644
> --- a/doc/general.texi
> +++ b/doc/general.texi
> @@ -53,14 +53,6 @@ instructions for installing the libraries.
>  Then pass @code{--enable-libopencore-amrnb} and/or
>  @code{--enable-libopencore-amrwb} to configure to enable them.
>
> - at subsection VisualOn AAC encoder library
> -
> -FFmpeg can make use of the VisualOn AACenc library for AAC encoding.
> -
> -Go to @url{http://sourceforge.net/projects/opencore-amr/} and follow the
> -instructions for installing the library.
> -Then pass @code{--enable-libvo-aacenc} to configure to enable it.
> -
>  @subsection VisualOn AMR-WB encoder library
>
>  FFmpeg can make use of the VisualOn AMR-WBenc library for AMR-WB encoding.
> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> index b9ffdb9..178bc18 100644
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -833,7 +833,6 @@ OBJS-$(CONFIG_LIBTHEORA_ENCODER)          +=
> libtheoraenc.o
>  OBJS-$(CONFIG_LIBTWOLAME_ENCODER)         += libtwolame.o
>  OBJS-$(CONFIG_LIBUTVIDEO_DECODER)         += libutvideodec.o
>  OBJS-$(CONFIG_LIBUTVIDEO_ENCODER)         += libutvideoenc.o
> -OBJS-$(CONFIG_LIBVO_AACENC_ENCODER)       += libvo-aacenc.o mpeg4audio.o
>  OBJS-$(CONFIG_LIBVO_AMRWBENC_ENCODER)     += libvo-amrwbenc.o
>  OBJS-$(CONFIG_LIBVORBIS_DECODER)          += libvorbisdec.o
>  OBJS-$(CONFIG_LIBVORBIS_ENCODER)          += libvorbisenc.o \
> diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c
> index 2128546..689106b 100644
> --- a/libavcodec/allcodecs.c
> +++ b/libavcodec/allcodecs.c
> @@ -575,7 +575,6 @@ void avcodec_register_all(void)
>      REGISTER_ENCODER(LIBTHEORA,         libtheora);
>      REGISTER_ENCODER(LIBTWOLAME,        libtwolame);
>      REGISTER_ENCDEC (LIBUTVIDEO,        libutvideo);
> -    REGISTER_ENCODER(LIBVO_AACENC,      libvo_aacenc);
>      REGISTER_ENCODER(LIBVO_AMRWBENC,    libvo_amrwbenc);
>      REGISTER_ENCDEC (LIBVORBIS,         libvorbis);
>      REGISTER_ENCDEC (LIBVPX_VP8,        libvpx_vp8);
> diff --git a/libavcodec/libvo-aacenc.c b/libavcodec/libvo-aacenc.c
> deleted file mode 100644
> index 7f21ad2..0000000
> --- a/libavcodec/libvo-aacenc.c
> +++ /dev/null
> @@ -1,200 +0,0 @@
> -/*
> - * AAC encoder wrapper
> - * Copyright (c) 2010 Martin Storsjo
> - *
> - * This file is part of FFmpeg.
> - *
> - * FFmpeg is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU Lesser General Public
> - * License as published by the Free Software Foundation; either
> - * version 2.1 of the License, or (at your option) any later version.
> - *
> - * FFmpeg is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> - * Lesser General Public License for more details.
> - *
> - * You should have received a copy of the GNU Lesser General Public
> - * License along with FFmpeg; if not, write to the Free Software
> - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
> 02110-1301 USA
> - */
> -
> -#include <vo-aacenc/voAAC.h>
> -#include <vo-aacenc/cmnMemory.h>
> -
> -#include "avcodec.h"
> -#include "audio_frame_queue.h"
> -#include "internal.h"
> -#include "mpeg4audio.h"
> -
> -#define FRAME_SIZE 1024
> -#define ENC_DELAY  1600
> -
> -typedef struct AACContext {
> -    VO_AUDIO_CODECAPI codec_api;
> -    VO_HANDLE handle;
> -    VO_MEM_OPERATOR mem_operator;
> -    VO_CODEC_INIT_USERDATA user_data;
> -    VO_PBYTE end_buffer;
> -    AudioFrameQueue afq;
> -    int last_frame;
> -    int last_samples;
> -} AACContext;
> -
> -
> -static int aac_encode_close(AVCodecContext *avctx)
> -{
> -    AACContext *s = avctx->priv_data;
> -
> -    s->codec_api.Uninit(s->handle);
> -    av_freep(&avctx->extradata);
> -    ff_af_queue_close(&s->afq);
> -    av_freep(&s->end_buffer);
> -
> -    return 0;
> -}
> -
> -static av_cold int aac_encode_init(AVCodecContext *avctx)
> -{
> -    AACContext *s = avctx->priv_data;
> -    AACENC_PARAM params = { 0 };
> -    int index, ret;
> -
> -    avctx->frame_size = FRAME_SIZE;
> -    avctx->initial_padding = ENC_DELAY;
> -    s->last_frame     = 2;
> -    ff_af_queue_init(avctx, &s->afq);
> -
> -    s->end_buffer = av_mallocz_array(avctx->channels, avctx->frame_size *
> 2);
> -    if (!s->end_buffer) {
> -        ret = AVERROR(ENOMEM);
> -        goto error;
> -    }
> -
> -    voGetAACEncAPI(&s->codec_api);
> -
> -    s->mem_operator.Alloc = cmnMemAlloc;
> -    s->mem_operator.Copy = cmnMemCopy;
> -    s->mem_operator.Free = cmnMemFree;
> -    s->mem_operator.Set = cmnMemSet;
> -    s->mem_operator.Check = cmnMemCheck;
> -    s->user_data.memflag = VO_IMF_USERMEMOPERATOR;
> -    s->user_data.memData = &s->mem_operator;
> -    s->codec_api.Init(&s->handle, VO_AUDIO_CodingAAC, &s->user_data);
> -
> -    params.sampleRate = avctx->sample_rate;
> -    params.bitRate    = avctx->bit_rate;
> -    params.nChannels  = avctx->channels;
> -    params.adtsUsed   = !(avctx->flags & AV_CODEC_FLAG_GLOBAL_HEADER);
> -    if (s->codec_api.SetParam(s->handle, VO_PID_AAC_ENCPARAM, &params)
> -        != VO_ERR_NONE) {
> -        av_log(avctx, AV_LOG_ERROR, "Unable to set encoding
> parameters\n");
> -        ret = AVERROR(EINVAL);
> -        goto error;
> -    }
> -
> -    for (index = 0; index < 16; index++)
> -        if (avctx->sample_rate == avpriv_mpeg4audio_sample_rates[index])
> -            break;
> -    if (index == 16) {
> -        av_log(avctx, AV_LOG_ERROR, "Unsupported sample rate %d\n",
> -                                    avctx->sample_rate);
> -        ret = AVERROR(ENOSYS);
> -        goto error;
> -    }
> -    if (avctx->flags & AV_CODEC_FLAG_GLOBAL_HEADER) {
> -        avctx->extradata_size = 2;
> -        avctx->extradata      = av_mallocz(avctx->extradata_size +
> -                                           AV_INPUT_BUFFER_PADDING_SIZE);
> -        if (!avctx->extradata) {
> -            ret = AVERROR(ENOMEM);
> -            goto error;
> -        }
> -
> -        avctx->extradata[0] = 0x02 << 3 | index >> 1;
> -        avctx->extradata[1] = (index & 0x01) << 7 | avctx->channels << 3;
> -    }
> -    return 0;
> -error:
> -    aac_encode_close(avctx);
> -    return ret;
> -}
> -
> -static int aac_encode_frame(AVCodecContext *avctx, AVPacket *avpkt,
> -                            const AVFrame *frame, int *got_packet_ptr)
> -{
> -    AACContext *s = avctx->priv_data;
> -    VO_CODECBUFFER input = { 0 }, output = { 0 };
> -    VO_AUDIO_OUTPUTINFO output_info = { { 0 } };
> -    VO_PBYTE samples;
> -    int ret;
> -
> -    /* handle end-of-stream small frame and flushing */
> -    if (!frame) {
> -        if (s->last_frame <= 0)
> -            return 0;
> -        if (s->last_samples > 0 && s->last_samples < ENC_DELAY -
> FRAME_SIZE) {
> -            s->last_samples = 0;
> -            s->last_frame--;
> -        }
> -        s->last_frame--;
> -        memset(s->end_buffer, 0, 2 * avctx->channels * avctx->frame_size);
> -        samples = s->end_buffer;
> -    } else {
> -        if (frame->nb_samples < avctx->frame_size) {
> -            s->last_samples = frame->nb_samples;
> -            memcpy(s->end_buffer, frame->data[0], 2 * avctx->channels *
> frame->nb_samples);
> -            samples = s->end_buffer;
> -        } else {
> -            samples = (VO_PBYTE)frame->data[0];
> -        }
> -        /* add current frame to the queue */
> -        if ((ret = ff_af_queue_add(&s->afq, frame)) < 0)
> -            return ret;
> -    }
> -
> -    if ((ret = ff_alloc_packet2(avctx, avpkt, FFMAX(8192, 768 *
> avctx->channels), 0)) < 0)
> -        return ret;
> -
> -    input.Buffer  = samples;
> -    input.Length  = 2 * avctx->channels * avctx->frame_size;
> -    output.Buffer = avpkt->data;
> -    output.Length = avpkt->size;
> -
> -    s->codec_api.SetInputData(s->handle, &input);
> -    if (s->codec_api.GetOutputData(s->handle, &output, &output_info)
> -        != VO_ERR_NONE) {
> -        av_log(avctx, AV_LOG_ERROR, "Unable to encode frame\n");
> -        return AVERROR(EINVAL);
> -    }
> -
> -    /* Get the next frame pts/duration */
> -    ff_af_queue_remove(&s->afq, avctx->frame_size, &avpkt->pts,
> -                       &avpkt->duration);
> -
> -    avpkt->size = output.Length;
> -    *got_packet_ptr = 1;
> -    return 0;
> -}
> -
> -/* duplicated from avpriv_mpeg4audio_sample_rates to avoid shared build
> - * failures */
> -static const int mpeg4audio_sample_rates[16] = {
> -    96000, 88200, 64000, 48000, 44100, 32000,
> -    24000, 22050, 16000, 12000, 11025, 8000, 7350
> -};
> -
> -AVCodec ff_libvo_aacenc_encoder = {
> -    .name           = "libvo_aacenc",
> -    .long_name      = NULL_IF_CONFIG_SMALL("Android VisualOn AAC
> (Advanced Audio Coding)"),
> -    .type           = AVMEDIA_TYPE_AUDIO,
> -    .id             = AV_CODEC_ID_AAC,
> -    .priv_data_size = sizeof(AACContext),
> -    .init           = aac_encode_init,
> -    .encode2        = aac_encode_frame,
> -    .close          = aac_encode_close,
> -    .supported_samplerates = mpeg4audio_sample_rates,
> -    .capabilities   = AV_CODEC_CAP_SMALL_LAST_FRAME | AV_CODEC_CAP_DELAY,
> -    .sample_fmts    = (const enum AVSampleFormat[]){ AV_SAMPLE_FMT_S16,
> -                                                     AV_SAMPLE_FMT_NONE },
> -};
> diff --git a/libavcodec/version.h b/libavcodec/version.h
> index 2b01e6f..3caebd8 100644
> --- a/libavcodec/version.h
> +++ b/libavcodec/version.h
> @@ -29,7 +29,7 @@
>  #include "libavutil/version.h"
>
>  #define LIBAVCODEC_VERSION_MAJOR  57
> -#define LIBAVCODEC_VERSION_MINOR  22
> +#define LIBAVCODEC_VERSION_MINOR  23
>  #define LIBAVCODEC_VERSION_MICRO 100
>
>  #define LIBAVCODEC_VERSION_INT  AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
> --
> 1.9.1
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list