]> git.sesse.net Git - vlc/commitdiff
Add a new aac audio encoder module based on the fdkaac library. It works correctly...
authorSergio Ammirata <sergio@ammirata.net>
Thu, 25 Oct 2012 15:10:07 +0000 (11:10 -0400)
committerIlkka Ollakka <ileoo@videolan.org>
Fri, 26 Oct 2012 10:10:08 +0000 (13:10 +0300)
Signed-off-by: Ilkka Ollakka <ileoo@videolan.org>
configure.ac
modules/codec/Modules.am
modules/codec/fdkaac.c [new file with mode: 0644]

index 5ca69b5fcc2150dd06e921cb824b594ec16508c6..c1ad99faeb90e8061c0ff1e929d2c8d663f1b753 100644 (file)
@@ -2486,6 +2486,11 @@ dnl twolame encoder plugin
 dnl
 PKG_ENABLE_MODULES_VLC([TWOLAME], [], [twolame], [MPEG Audio Layer 2 encoder], [auto], [], [], [ -DLIBTWOLAME_STATIC])
 
+dnl
+dnl fdk-aac encoder plugin
+dnl
+PKG_ENABLE_MODULES_VLC([FDKAAC], [], [fdk-aac], [FDK-AAC encoder], [disabled])
+
 dnl
 dnl  QuickTime plugin
 dnl
index aa71726e730d8228f1a0c62ffd92f7a1793a52c9..d78ffb4c93b57d569eb557871808c6614369b0c1 100644 (file)
@@ -55,6 +55,7 @@ SOURCES_stl = stl.c
 SOURCES_dmo = dmo/dmo.c dmo/dmo.h dmo/buffer.c
 SOURCES_ddummy = ddummy.c
 SOURCES_edummy = edummy.c
+SOURCES_fdkaac = fdkaac.c
 
 libvlc_LTLIBRARIES += \
        liba52_plugin.la \
diff --git a/modules/codec/fdkaac.c b/modules/codec/fdkaac.c
new file mode 100644 (file)
index 0000000..bc5220e
--- /dev/null
@@ -0,0 +1,537 @@
+/*****************************************************************************
+ * aac.c: FDK-AAC Encoder plugin for vlc.
+ *****************************************************************************
+ * Copyright (C) 2012 Sergio Ammirata
+ *
+ * Authors: Sergio Ammirata <sergio@ammirata.net>
+ *
+ * This library 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.
+ * 
+ * This library 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 this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ *****************************************************************************/
+
+/*****************************************************************************
+ * Preamble
+ *****************************************************************************/
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+#include <fdk-aac/aacenc_lib.h>
+
+#include <vlc_common.h>
+#include <vlc_plugin.h>
+#include <vlc_codec.h>
+
+static int OpenEncoder( vlc_object_t * );
+static void CloseEncoder( vlc_object_t * );
+
+#define ENC_CFG_PREFIX "sout-fdkaac-"
+
+#define AOT_TEXT N_("Encoder Profile")
+#define AOT_LONGTEXT N_( "Encoder Algorithm to use" )
+
+#define SIDEBAND_TEXT N_("Enable spectral band replication")
+#define SIDEBAND_LONGTEXT N_( "This is an optional feature only for the AAC-ELD profile" )
+
+#define VBR_QUALITY_TEXT N_("VBR Quality")
+#define VBR_QUALITY_LONGTEXT N_( "Quality of the VBR Encoding (0=cbr, 1-5 constant quality vbr, 5 is best" )
+
+#define AFTERBURNER_TEXT N_("Enable afterburner library")
+#define AFTERBURNER_LONGTEXT N_( "This library will produce higher quality audio at the expense of additional CPU usage (default is enabled)" )
+
+#define BITRATE_TEXT N_("CBR Bitrate")
+#define BITRATE_LONGTEXT N_( "Bitrate of desired stream (in bps)" )
+
+#define SIGNALING_TEXT N_("Signaling mode of the extension AOT")
+#define SIGNALING_LONGTEXT N_( "1 is explicit for SBR and implicit for PS (default), 2 is explicit hierarchical" )
+
+#define  CH_ORDER_MPEG 0  /*!< MPEG channel ordering (e. g. 5.1: C, L, R, SL, SR, LFE)           */
+#define  CH_ORDER_WAV 1   /*!< WAV fileformat channel ordering (e. g. 5.1: L, R, C, LFE, SL, SR) */
+#define  CH_ORDER_WG4 2   /*!< WG4 fileformat channel ordering (e. g. 5.1: L, R, SL, SR, C, LFE) */
+
+#define PROFILE_AAC_LC 2
+#define PROFILE_AAC_HE 5
+#define PROFILE_AAC_HE_v2 29
+#define PROFILE_AAC_LD 23
+#define PROFILE_AAC_ELD 39
+
+#define SIGNALING_COMPATIBLE 1
+#define SIGNALING_HIERARCHICAL 2
+
+static const int pi_aot_values[] = { PROFILE_AAC_LC, PROFILE_AAC_HE, PROFILE_AAC_HE_v2, PROFILE_AAC_LD, PROFILE_AAC_ELD };
+static const char *const ppsz_aot_descriptions[] =
+{ N_("AAC-LC"), N_("HE-AAC"), N_("HE-AAC-v2"), N_("AAC-LD"), N_("AAC-ELD") };
+
+vlc_module_begin ()
+    set_shortname( N_("FDKAAC") )
+    set_description( N_("FDK-AAC Audio encoder") )
+    set_capability( "encoder", 50 )
+    set_callbacks( OpenEncoder, CloseEncoder )
+    add_shortcut( "fdkaac" )
+    set_category( CAT_INPUT )
+    set_subcategory( SUBCAT_INPUT_ACODEC )
+    add_integer( ENC_CFG_PREFIX "profile", PROFILE_AAC_LC, AOT_TEXT,
+             AOT_LONGTEXT, false )
+    change_integer_list( pi_aot_values, ppsz_aot_descriptions );
+    add_bool( ENC_CFG_PREFIX "sbr", false, SIDEBAND_TEXT,
+              SIDEBAND_LONGTEXT, false )
+    add_integer( ENC_CFG_PREFIX "vbr", 0, VBR_QUALITY_TEXT, 
+              VBR_QUALITY_LONGTEXT, false )
+    change_integer_range (0, 5)
+    add_bool( ENC_CFG_PREFIX "afterburner", true, AFTERBURNER_TEXT,
+              AFTERBURNER_LONGTEXT, true )
+    add_integer( ENC_CFG_PREFIX "signaling", SIGNALING_COMPATIBLE, SIGNALING_TEXT,
+             SIGNALING_LONGTEXT, true )
+    change_integer_range (0, 2)
+vlc_module_end ()
+
+/*****************************************************************************
+ * Local prototypes
+ *****************************************************************************/
+static block_t *EncodeAudio( encoder_t *p_enc, block_t *p_buf );
+
+static const char *const ppsz_enc_options[] = {
+    "profile", "sbr", "vbr", "afterburner", "signaling", NULL
+};
+
+/*****************************************************************************
+ * encoder_sys_t : aac encoder descriptor
+ *****************************************************************************/
+struct encoder_sys_t
+{
+    double d_compression_ratio;
+    mtime_t i_pts_last;
+    int i_aot; /* This stores the aac profile chosen */
+    int i_vbr; /* cbr or vbr-quality value chosen */
+    int i_signaling; /* Library feature for backwards compatibility */
+    int i_encoderdelay; /* Samples delay introduced by the profile */
+    int i_frame_size;
+    int i_maxoutputsize; /* Maximum buffer size for encoded output */
+    HANDLE_AACENCODER handle;
+    bool b_afterburner; /* Library feature for additional quality */
+    bool b_eld_sbr; /* Spectral band replication option for ELD profile */
+};
+
+static const char *aac_get_errorstring(AACENC_ERROR erraac)
+{
+    switch (erraac) {
+    case AACENC_OK:
+        return "No error";
+    case AACENC_INVALID_HANDLE:
+        return "Invalid handle";
+    case AACENC_MEMORY_ERROR:
+        return "Memory allocation error";
+    case AACENC_UNSUPPORTED_PARAMETER:
+        return "Unsupported parameter";
+    case AACENC_INVALID_CONFIG:
+        return "Invalid config";
+    case AACENC_INIT_ERROR:
+        return "Initialization error";
+    case AACENC_INIT_AAC_ERROR:
+        return "AAC library initialization error";
+    case AACENC_INIT_SBR_ERROR:
+        return "SBR library initialization error";
+    case AACENC_INIT_TP_ERROR:
+        return "Transport library initialization error";
+    case AACENC_INIT_META_ERROR:
+        return "Metadata library initialization error";
+    case AACENC_ENCODE_ERROR:
+        return "Encoding error";
+    case AACENC_ENCODE_EOF:
+        return "End of file";
+    default:
+        return "Unknown error";
+    }
+}
+
+/*****************************************************************************
+ * OpenDecoder: open the encoder.
+ *****************************************************************************/
+static int OpenEncoder( vlc_object_t *p_this )
+{
+    encoder_t *p_enc;
+    encoder_sys_t *p_sys;
+    CHANNEL_MODE mode;
+    AACENC_ERROR erraac;
+    bool b_profile_selected;
+    int sce;
+    int cpe;
+    int i_profile;
+    int i_bitrate;
+
+    p_enc = (encoder_t *)p_this;
+    b_profile_selected = false;
+    sce = 0;
+    cpe = 0;
+
+    if( p_enc->fmt_out.i_codec != VLC_FOURCC( 'l', 'a', 'a', 'c' ) && 
+        p_enc->fmt_out.i_codec != VLC_FOURCC( 'h', 'a', 'a', 'c' ) && 
+        p_enc->fmt_out.i_codec != VLC_FOURCC( 's', 'a', 'a', 'c' ) &&
+        p_enc->fmt_out.i_codec != VLC_CODEC_MP4A )
+    {
+        return VLC_EGENERIC;
+    }
+    else if ( p_enc->fmt_out.i_codec == VLC_FOURCC( 'l', 'a', 'a', 'c' ) )
+    {
+        b_profile_selected = true;
+        i_profile = PROFILE_AAC_LC;
+    }
+    else if ( p_enc->fmt_out.i_codec == VLC_FOURCC( 'h', 'a', 'a', 'c' ) )
+    {
+        b_profile_selected = true;
+        i_profile = PROFILE_AAC_HE;
+    }
+    else if ( p_enc->fmt_out.i_codec == VLC_FOURCC( 's', 'a', 'a', 'c' ) )
+    {
+        b_profile_selected = true;
+        i_profile = PROFILE_AAC_HE_v2;
+    }
+
+    switch (p_enc->fmt_in.audio.i_channels) {
+    case 1: mode = MODE_1;       sce = 1; cpe = 0; break;
+    case 2: mode = MODE_2;       sce = 0; cpe = 1; break;
+    case 3: mode = MODE_1_2;     sce = 1; cpe = 1; break;
+    case 4: mode = MODE_1_2_1;   sce = 2; cpe = 1; break;
+    case 5: mode = MODE_1_2_2;   sce = 1; cpe = 2; break;
+    case 6: mode = MODE_1_2_2_1; sce = 2; cpe = 2; break;
+    case 8: mode = MODE_1_2_2_2_1; sce = 2; cpe = 3; break;
+    default:
+        msg_Err( p_enc, "we do not support > 8 input channels, this input has %i", 
+                        p_enc->fmt_in.audio.i_channels );
+        return VLC_EGENERIC;
+    }
+
+    msg_Info(p_enc, "Initializing AAC Encoder, %i channels", p_enc->fmt_in.audio.i_channels);
+
+    /* Allocate the memory needed to store the encoder's structure */
+    p_sys = (encoder_sys_t *)malloc(sizeof(encoder_sys_t));
+    if( unlikely( !p_sys ) )
+        return VLC_ENOMEM;
+    p_enc->p_sys = p_sys;
+    p_enc->fmt_in.i_codec = VLC_CODEC_S16N;
+    p_enc->fmt_out.i_cat = AUDIO_ES;
+    p_enc->fmt_out.i_codec = VLC_CODEC_MP4A;
+
+    config_ChainParse( p_enc, ENC_CFG_PREFIX, ppsz_enc_options, p_enc->p_cfg );
+
+    if ( b_profile_selected == false )
+        p_sys->i_aot = var_InheritInteger( p_enc, ENC_CFG_PREFIX "profile" );
+    else
+        p_sys->i_aot = i_profile;
+    p_sys->b_eld_sbr = var_InheritBool( p_enc, ENC_CFG_PREFIX "sbr" );
+    p_sys->i_vbr = var_InheritInteger( p_enc, ENC_CFG_PREFIX "vbr" );
+    p_sys->b_afterburner = var_InheritBool( p_enc, ENC_CFG_PREFIX "afterburner" );
+    p_sys->i_signaling = var_InheritInteger( p_enc, ENC_CFG_PREFIX "signaling" );
+    p_sys->i_pts_last = 0;
+
+    if ((p_sys->i_aot == PROFILE_AAC_HE || p_sys->i_aot == PROFILE_AAC_HE_v2) && p_sys->i_vbr > 3)
+    {
+        msg_Warn(p_enc, "Maximum VBR quality for this profile is 3, setting vbr=3");
+        p_sys->i_vbr = 3;
+    }
+    if ((erraac = aacEncOpen(&p_sys->handle, 0, p_enc->fmt_in.audio.i_channels)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to open encoder: %s", aac_get_errorstring(erraac));
+        free( p_sys );
+        return VLC_EGENERIC;
+    }
+    if ( p_sys->i_aot == PROFILE_AAC_HE_v2 && p_enc->fmt_in.audio.i_channels != 2 )
+    {
+        msg_Err(p_enc, "The HE-AAC-v2 profile can only be used with stereo sources");
+        goto error;
+    }
+    if ( p_sys->i_aot == PROFILE_AAC_ELD && p_enc->fmt_in.audio.i_channels != 2 )
+    {
+        msg_Err(p_enc, "The ELD-AAC profile can only be used with stereo sources");
+        goto error;
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_AOT, p_sys->i_aot)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the Profile %i: %s", p_sys->i_aot, aac_get_errorstring(erraac));
+        goto error;
+    }
+    if (p_sys->i_aot == PROFILE_AAC_ELD && p_sys->b_eld_sbr) {
+        if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_SBR_MODE, 1)) != AACENC_OK) {
+            msg_Err(p_enc, "Unable to set SBR mode for ELD: %s", aac_get_errorstring(erraac));
+        goto error;
+        }
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_SAMPLERATE, 
+                    p_enc->fmt_out.audio.i_rate)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the sample rate %i: %s",p_enc->fmt_out.audio.i_rate,
+                        aac_get_errorstring(erraac));
+        goto error;
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_CHANNELMODE, mode)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the channel mode: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_CHANNELORDER, CH_ORDER_WG4)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the sound channel order: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    if (p_sys->i_vbr != 0) {
+        if ((erraac = aacEncoder_SetParam(p_sys->handle, 
+                         AACENC_BITRATEMODE, p_sys->i_vbr)) != AACENC_OK) {
+            msg_Err(p_enc, "Unable to set the VBR bitrate mode: %s", aac_get_errorstring(erraac));
+            goto error;
+        }
+    } else {
+        if (p_enc->fmt_out.i_bitrate == 0) {
+            if (p_sys->i_aot == PROFILE_AAC_HE_v2) {
+                sce = 1;
+                cpe = 0;
+            }
+            i_bitrate = (96*sce + 128*cpe) * p_enc->fmt_out.audio.i_rate / 44;
+            if (p_sys->i_aot == PROFILE_AAC_HE ||
+                p_sys->i_aot == PROFILE_AAC_HE_v2 ||
+                p_sys->b_eld_sbr)
+                i_bitrate /= 2;
+            p_enc->fmt_out.i_bitrate = i_bitrate;
+            msg_Info(p_enc, "Setting optimal bitrate of %i", i_bitrate);
+        }
+        else
+        {
+            i_bitrate = p_enc->fmt_out.i_bitrate;
+        }
+        if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_BITRATE, 
+                                         i_bitrate)) != AACENC_OK) {
+            msg_Err(p_enc, "Unable to set the bitrate %i: %s", i_bitrate,
+                    aac_get_errorstring(erraac));
+            goto error;
+        }
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_TRANSMUX, 0)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the ADTS transmux: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_SIGNALING_MODE, 
+                                 (int)p_sys->i_signaling)) != AACENC_OK) {
+      /* use explicit backward compatible =1 */
+      /* use explicit hierarchical signaling =2 */
+        msg_Err(p_enc, "Unable to set signaling mode: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    if ((erraac = aacEncoder_SetParam(p_sys->handle, AACENC_AFTERBURNER, 
+                               (int)p_sys->b_afterburner)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to set the afterburner mode: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    if ((erraac = aacEncEncode(p_sys->handle, NULL, NULL, NULL, NULL)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to initialize the encoder: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+    AACENC_InfoStruct info = { 0 };
+    if ((erraac = aacEncInfo(p_sys->handle, &info)) != AACENC_OK) {
+        msg_Err(p_enc, "Unable to get the encoder info: %s", aac_get_errorstring(erraac));
+        goto error;
+    }
+
+    /* The maximum packet size is 6144 bits aka 768 bytes per channel. */
+    p_sys->i_maxoutputsize = 768*p_enc->fmt_in.audio.i_channels;
+    p_enc->fmt_in.audio.i_bitspersample = 16;
+    p_sys->i_frame_size = info.frameLength;
+    p_sys->i_encoderdelay = info.encoderDelay;
+
+    p_enc->fmt_out.i_extra = info.confSize;
+    if( p_enc->fmt_out.i_extra )
+    {
+        p_enc->fmt_out.p_extra = malloc( p_enc->fmt_out.i_extra );
+        if ( p_enc->fmt_out.p_extra == NULL )
+        {
+            msg_Err(p_enc, "Unable to allocate fmt_out.p_extra");
+            goto error;
+        }
+        memcpy( p_enc->fmt_out.p_extra, info.confBuf,
+                p_enc->fmt_out.i_extra );
+    }
+
+    p_enc->pf_encode_audio = EncodeAudio;
+
+#ifndef NDEBUG
+    // TODO: Add more debug info to this config printout
+    msg_Dbg(p_enc, "fmt_out.p_extra = %i", p_enc->fmt_out.i_extra);
+#endif
+
+    return VLC_SUCCESS;
+
+error:
+    aacEncClose(&p_sys->handle);
+    free( p_sys );
+    return VLC_EGENERIC;
+}
+
+/****************************************************************************
+ * EncodeAudio: the whole thing
+ ****************************************************************************/
+static block_t *EncodeAudio( encoder_t *p_enc, block_t *p_aout_buf )
+{
+    encoder_sys_t *p_sys;
+    int16_t *p_buffer;
+    int i_samples;
+    int i_loop_count;
+    int i_samples_left;
+    mtime_t i_pts_out;
+    block_t *p_chain;
+    AACENC_ERROR erraac;
+
+    p_sys = p_enc->p_sys;
+    p_chain = NULL;
+
+    if ( likely( p_aout_buf ) )
+    {
+        p_buffer = (int16_t *)p_aout_buf->p_buffer;
+        i_samples = p_aout_buf->i_nb_samples;
+        i_pts_out = p_aout_buf->i_pts - (mtime_t)((double)CLOCK_FREQ *
+               (double)p_sys->i_encoderdelay /
+               (double)p_enc->fmt_out.audio.i_rate);
+        if (p_sys->i_pts_last == 0)
+            p_sys->i_pts_last = i_pts_out - (mtime_t)((double)CLOCK_FREQ *
+               (double)(p_sys->i_frame_size) /
+               (double)p_enc->fmt_out.audio.i_rate);
+    }
+    else
+    {
+        i_samples = 0;
+        i_pts_out = p_sys->i_pts_last;
+    }
+
+    i_samples_left = i_samples;
+    i_loop_count = 0;
+
+    while ( i_samples_left >= 0 )
+    {
+        AACENC_BufDesc in_buf = { 0 }, out_buf = { 0 };
+        AACENC_InArgs in_args = { 0 };
+        AACENC_OutArgs out_args = { 0 };
+        int in_identifier = IN_AUDIO_DATA;
+        int in_size, in_elem_size;
+        int out_identifier = OUT_BITSTREAM_DATA;
+        int out_size, out_elem_size;
+        void *in_ptr, *out_ptr;
+
+        if ( unlikely(i_samples == 0) ) {
+            // this forces the encoder to purge whatever is left in the internal buffer
+            in_args.numInSamples = -1;
+        } else {
+            in_ptr = p_buffer + (i_samples - i_samples_left)*p_enc->fmt_in.audio.i_channels;
+            in_size = 2*p_enc->fmt_in.audio.i_channels*i_samples_left;
+            in_elem_size = 2;
+            in_args.numInSamples = p_enc->fmt_in.audio.i_channels*i_samples_left;
+            in_buf.numBufs = 1;
+            in_buf.bufs = &in_ptr;
+            in_buf.bufferIdentifiers = &in_identifier;
+            in_buf.bufSizes = &in_size;
+            in_buf.bufElSizes = &in_elem_size;
+        }
+        block_t *p_block;
+        p_block = block_New( p_enc, p_sys->i_maxoutputsize );
+        p_block->i_buffer = p_sys->i_maxoutputsize;
+        out_ptr = p_block->p_buffer;
+        out_size = p_block->i_buffer;
+        out_elem_size = 1;
+        out_buf.numBufs = 1;
+        out_buf.bufs = &out_ptr;
+        out_buf.bufferIdentifiers = &out_identifier;
+        out_buf.bufSizes = &out_size;
+        out_buf.bufElSizes = &out_elem_size;
+
+        if ((erraac = aacEncEncode(p_sys->handle, &in_buf, &out_buf, &in_args, &out_args)) != AACENC_OK) {
+            if (erraac == AACENC_ENCODE_EOF) {
+                msg_Info( p_enc, "Encoding final bytes (EOF)");
+            }
+            else
+            {
+                msg_Err( p_enc, "Encoding failed: %s", aac_get_errorstring(erraac));
+                block_Release(p_block);
+                break;
+            }
+        }
+        if ( out_args.numOutBytes > 0 )
+        {
+            p_block->i_buffer = out_args.numOutBytes;
+            if ( unlikely(i_samples == 0) )
+            {
+                // I only have the numOutBytes so approximate based on compression factor
+                double d_samples_forward = p_sys->d_compression_ratio*(double)out_args.numOutBytes;
+                i_pts_out += (mtime_t)d_samples_forward;
+                p_block->i_length = (mtime_t)d_samples_forward;
+                // TODO: It would be more precise (a few microseconds) to use d_samples_forward = 
+                // (mtime_t)CLOCK_FREQ * (mtime_t)p_sys->i_frame_size/(mtime_t)p_enc->fmt_out.audio.i_rate
+                // but I am not sure if the lib always outputs a full frame when 
+                // emptying the internal buffer in the EOF scenario
+            }
+            else
+            {
+                if ( i_loop_count == 0 )
+                {
+                    // There can be an implicit delay in the first loop cycle because leftover bytes
+                    // in the library buffer from the prior block
+                    double d_samples_delay = (double)p_sys->i_frame_size - (double)out_args.numInSamples /
+                                             (double)p_enc->fmt_in.audio.i_channels;
+                    i_pts_out -= (mtime_t)((double)CLOCK_FREQ * d_samples_delay /
+                                           (double)p_enc->fmt_out.audio.i_rate);
+                    //p_block->i_length = (mtime_t)((double)CLOCK_FREQ * (double)p_sys->i_frame_size / 
+                    //    (double)p_enc->fmt_out.audio.i_rate);
+                    p_block->i_length = i_pts_out - p_sys->i_pts_last;
+                }
+                else
+                {
+                    double d_samples_forward = (double)out_args.numInSamples/(double)p_enc->fmt_in.audio.i_channels;
+                    double d_length = ((double)CLOCK_FREQ * d_samples_forward /
+                                            (double)p_enc->fmt_out.audio.i_rate);
+                    i_pts_out += (mtime_t) d_length;
+                    p_block->i_length = (mtime_t) d_length;
+                }
+            }
+            p_block->i_dts = p_block->i_pts = i_pts_out;
+            block_ChainAppend( &p_chain, p_block );
+            //msg_Dbg( p_enc, "p_block->i_dts %llu, p_block->i_length %llu, p_sys->i_pts_last %llu\n\ 
+            //                  out_args.numOutBytes = %i, out_args.numInSamples = %i, i_samples %i, i_loop_count %i",
+            //                  p_block->i_dts , p_block->i_length,p_sys->i_pts_last,
+            //                  out_args.numOutBytes, out_args.numInSamples, i_samples, i_loop_count);
+            if ( likely(i_samples > 0) )
+            {
+                p_sys->d_compression_ratio = (double)p_block->i_length / (double)out_args.numOutBytes;
+                i_samples_left -= out_args.numInSamples/p_enc->fmt_in.audio.i_channels;
+                p_sys->i_pts_last = i_pts_out;
+            }
+        }
+        else
+        {
+            block_Release(p_block);
+            //msg_Dbg( p_enc, "aac_encode_audio: not enough data yet");
+            break;
+        }
+        if ( unlikely(i_loop_count++ > 100) )
+        {
+            msg_Err( p_enc, "Loop count greater than 100!!!, something must be wrong with the encoder library");
+            break;
+        }
+    }
+
+    return p_chain;
+
+}
+
+/*****************************************************************************
+ * CloseDecoder: decoder destruction
+ *****************************************************************************/
+static void CloseEncoder( vlc_object_t *p_this )
+{
+    encoder_t *p_enc = (encoder_t *)p_this;
+    encoder_sys_t *p_sys = p_enc->p_sys;
+
+    aacEncClose(&p_sys->handle);
+
+    free( p_sys );
+}