On Fri, 20 May 2011, Ronald S. Bultje wrote:
> On Fri, May 20, 2011 at 8:14 AM, Martin Storsjö <[email protected]> wrote:
> > This patchset implements optional MP4A-LATM RTP payload
> > by adding a new encoders that output CODEC_ID_AAC_LATM.
> > This makes the rtpenc/sdp part really trivial, and the
> > additions to the existing AAC encoders arguably is quite
> > small.
> >
> > I do feel that this is kinda ugly though, having to add extra
> > codecs for something that really is just a different way of
> > muxing the same raw data.
>
> I agree it's not necessarily nice. But yet I do prefer this approach,
> since we're using the same approach for the demuxers also.
>
> I'd prefer to not have to add codec-specific avoptions (or flags, or
> anything really) to lavf unless there really is no other way...
Ah crap, I came to think of one downside to this approach, which is kind
of a showstopper for me. If I create an mp4 with RTP hinting, this
approach doesn't work, since I can't input AAC_LATM to the mp4 muxer - I
need to write normal AAC to the main audio track, and enable the LATM
format only for the RTP hint track. That also rules out bitstream filters.
Hence, this approach might not be viable after all...
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
