On Wed, Feb 14, 2024 at 11:31:21PM +0000, Mark Thompson wrote: > On 13/02/2024 22:50, Matthieu Bouron wrote: > > This will allow to use the jni utils in libavformat. This will be mostly > > useful > > to add Android content-uri support. > > > > This deprecates avcodec/jni.h functions in favor of the ones from > > avutil/jni.h. > > --- > > doc/APIchanges | 6 + > > libavcodec/Makefile | 3 +- > > libavcodec/jni.c | 48 +---- > > libavcodec/jni.h | 8 + > > libavcodec/mediacodec.c | 6 +- > > libavcodec/mediacodec_surface.c | 6 +- > > libavcodec/mediacodec_wrapper.c | 200 ++++++++++----------- > > libavcodec/mediacodecdec.c | 3 +- > > libavutil/Makefile | 4 + > > libavutil/jni.c | 78 ++++++++ > > libavutil/jni.h | 46 +++++ > > libavcodec/ffjni.c => libavutil/jniutils.c | 36 ++-- > > libavcodec/ffjni.h => libavutil/jniutils.h | 26 ++- > > 13 files changed, 283 insertions(+), 187 deletions(-) > > create mode 100644 libavutil/jni.c > > create mode 100644 libavutil/jni.h > > rename libavcodec/ffjni.c => libavutil/jniutils.c (88%) > > rename libavcodec/ffjni.h => libavutil/jniutils.h (84%) > > Why? > > libavformat already depends on libavcodec.
Since both libavformat and libavcodec will use it, it seems more logical to put the code in libavutil. It could also benefits libavutil/hwcontext_mediacodec. [...] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
