Quoting Zhao Zhili (2024-08-26 13:52:41)
On Aug 24, 2024, at 22:40, Matthieu Bouron <matthieu.bouron@gmail.com> wrote:
---
Diff with the v1: - dropped support of ENCODING_PCM_24BIT_PACKED as it uses 3 consecutive bytes for one sample and there is no direct AVSampleFormat equivalent (that would require a dedicated copy routine to copy the 3 bytes into the 4 bytes provided by AV_SAMPLE_FMT_S32). I believe this is something that can be added later on. - Addressed all comments except the one about the assert failure using `/ffmpeg_g -c:a opus_mediacodec -i /sdcard/opus.mp4 -f null -`. Maybe it is fixed by your recent patch related to the draining state of the decoder ?
I still get the same assert failure with and only with opus
Why should it support opus again? Or flac or any other patent-unecumbered codec? I thought that is the plan, given that patents is the only reason given for these wrappers. -- Anton Khirnov