On Jul 16, 2024, at 21:20, Matthieu Bouron <matthieu.bouron@gmail.com> wrote:
On Wed, Jul 10, 2024 at 6:31 PM Matthieu Bouron <matthieu.bouron@gmail.com> wrote:
On Wed, Jul 10, 2024 at 4:04 PM Zhao Zhili <quinkblack@foxmail.com> wrote:
On Jun 12, 2024, at 21:42, Matthieu Bouron <matthieu.bouron@gmail.com> wrote:
Hello,
This patchset adds Mediacodec audio decoders support. Currently, only AAC, AMR, MP3, FLAC, VORBIS and OPUS are supported.
This is mainly useful to avoid shipping Android builds of FFmpeg that are subjects to licensing/patents (due to AAC and AMR).
I’m not keen on put OS audio decoder/encoder wrapper into FFmpeg. They don’t bring new features, they don’t improve performance. I know these type of wrapper exist in current project, but I’m not sure if it’s a good idea to add more.
I agree that on the technical side it doesn't bring new features nor performance improvements. It's all about avoiding licensing/patents issues with AAC and AMR here. In this specific case we already have the wrapper infrastructure, the audio part only needs small adjustments to work. Moreover, if that helps, I can reduce the scope of the patch to AAC and AMR only and get rid of mp3/flac/vorbis/opus support. What do you think ?
Ping.
IMHO, this benefits users wanting to ship an Android app that relies on FFmpeg upstream in countries that are subject to AAC/AMR licensing. While I agree that it's not great from a purely technical pov since we already have better native decoders, it will allow the use of FFmpeg in such situation without the need to use or create another FFmpeg fork dedicated to Android. Plus, as I said above, we already have the wrapper and the additional code to make it work for audio is relatively small and scoped. Restricting the wrapper to AAC/AMR seems like a good compromise to me.
Sounds reasonable, but I’d like to get more opinions. cc TC.
[...] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".