On Fri, Sep 23, 2011 at 7:13 PM, Alex Converse <[email protected]> wrote: > 2011/9/23 Måns Rullgård <[email protected]>: >> Kieran Kunhya <[email protected]> writes: >> >>> On Thu, Sep 22, 2011 at 8:22 PM, Janne Grunau <[email protected]> >>> wrote: >>>> >>>> The LATM parser expose the same extradata but has the bitstream still is >>>> LATM encapsulated. >>>> --- >>>> libavformat/latmenc.c | 5 +++++ >>>> 1 files changed, 5 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/libavformat/latmenc.c b/libavformat/latmenc.c >>>> index 8d67b5a..aecffe2 100644 >>>> --- a/libavformat/latmenc.c >>>> +++ b/libavformat/latmenc.c >>>> @@ -75,6 +75,11 @@ static int latm_write_header(AVFormatContext *s) >>>> LATMContext *ctx = s->priv_data; >>>> AVCodecContext *avctx = s->streams[0]->codec; >>>> >>>> + if (avctx->codec_id != CODEC_ID_AAC) { >>>> + av_log(s, AV_LOG_ERROR, "LATM/LOAS is only specified for AAC >>>> data.\n"); >>>> + return AVERROR_INVALIDDATA; >>>> + } >>>> + >>>> if (avctx->extradata_size > 0 && >>>> latm_decode_extradata(ctx, avctx->extradata, >>>> avctx->extradata_size) < 0) >>>> return AVERROR_INVALIDDATA; >>>> -- >>>> 1.7.6.1 >>>> >>> >>> ALS is also allowed in LATM/LOAS >> >> ALS is part of the AAC spec, no? >> > > ALS is part of 14496-3 but not 13818-7. 14496-3 doesn't claim to be a > specification for AAC unlike 13818-7 but instead "Audio." > > Based on my reading of LATM any codec that is assigned one or more > AOTs by 14496-3 (including mpeg surround which is also not described > in 14496-3 but still given an AOT) is allowed in LATM.
Whilst the above is true, my point was that AAC and ALS have two separate CODEC_IDs in libav. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
