* [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method
@ 2023-01-09 16:36 Zhao Zhili
2023-01-09 12:32 ` Tomas Härdin
0 siblings, 1 reply; 4+ messages in thread
From: Zhao Zhili @ 2023-01-09 16:36 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Zhao Zhili
From: Zhao Zhili <zhilizhao@tencent.com>
android_get_device_api_level() is a static inline method before API
level 29. It was implemented via __system_property_get(). We can do
the same thing, but I don't want to mess up with
__system_property_get.
Signed-off-by: Zhao Zhili <zhilizhao@tencent.com>
---
libavcodec/mediacodec_wrapper.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c
index 4d6e9487b8..b675623a0b 100644
--- a/libavcodec/mediacodec_wrapper.c
+++ b/libavcodec/mediacodec_wrapper.c
@@ -2513,6 +2513,10 @@ FFAMediaCodec* ff_AMediaCodec_createEncoderByType(const char *mime_type, int ndk
int ff_Build_SDK_INT(AVCodecContext *avctx)
{
int ret = -1;
+
+#if __ANDROID_API__ >= 24
+ ret = android_get_device_api_level();
+#else
JNIEnv *env = NULL;
jclass versionClass;
jfieldID sdkIntFieldID;
@@ -2522,5 +2526,8 @@ int ff_Build_SDK_INT(AVCodecContext *avctx)
sdkIntFieldID = (*env)->GetStaticFieldID(env, versionClass, "SDK_INT", "I");
ret = (*env)->GetStaticIntField(env, versionClass, sdkIntFieldID);
(*env)->DeleteLocalRef(env, versionClass);
+#endif
+ av_log(avctx, AV_LOG_DEBUG, "device api level %d\n", ret);
+
return ret;
}
--
2.25.1
_______________________________________________
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".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method
2023-01-09 16:36 [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method Zhao Zhili
@ 2023-01-09 12:32 ` Tomas Härdin
2023-01-09 14:41 ` Zhao Zhili
0 siblings, 1 reply; 4+ messages in thread
From: Tomas Härdin @ 2023-01-09 12:32 UTC (permalink / raw)
To: FFmpeg development discussions and patches
tis 2023-01-10 klockan 00:36 +0800 skrev Zhao Zhili:
> From: Zhao Zhili <zhilizhao@tencent.com>
>
> android_get_device_api_level() is a static inline method before API
> level 29. It was implemented via __system_property_get(). We can do
> the same thing, but I don't want to mess up with
> __system_property_get.
The code tests on API >= 24 not 29. Or do you mean that
android_get_device_api_level() is available since 24 but not used until
29?
/Tomas
_______________________________________________
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".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method
2023-01-09 12:32 ` Tomas Härdin
@ 2023-01-09 14:41 ` Zhao Zhili
2023-01-09 16:44 ` Tomas Härdin
0 siblings, 1 reply; 4+ messages in thread
From: Zhao Zhili @ 2023-01-09 14:41 UTC (permalink / raw)
To: 'FFmpeg development discussions and patches'
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Tomas Härdin
> Sent: 2023年1月9日 20:32
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method
>
> tis 2023-01-10 klockan 00:36 +0800 skrev Zhao Zhili:
> > From: Zhao Zhili <zhilizhao@tencent.com>
> >
> > android_get_device_api_level() is a static inline method before API
> > level 29. It was implemented via __system_property_get(). We can do
> > the same thing, but I don't want to mess up with
> > __system_property_get.
>
> The code tests on API >= 24 not 29. Or do you mean that
> android_get_device_api_level() is available since 24 but not used until
> 29?
If android_get_device_api_level() is a global symbol instead of inline method, I would
use dlsym() instead to make it works on build target api level < 24 and device/runtime
api level >= 24.
>
>
> /Tomas
>
> _______________________________________________
> 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".
_______________________________________________
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".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method
2023-01-09 14:41 ` Zhao Zhili
@ 2023-01-09 16:44 ` Tomas Härdin
0 siblings, 0 replies; 4+ messages in thread
From: Tomas Härdin @ 2023-01-09 16:44 UTC (permalink / raw)
To: FFmpeg development discussions and patches
mån 2023-01-09 klockan 22:41 +0800 skrev Zhao Zhili:
>
> > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> > Tomas Härdin
> > Sent: 2023年1月9日 20:32
> > To: FFmpeg development discussions and patches <
> > ffmpeg-devel@ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper:
> > get api level via NDK method
> >
> > tis 2023-01-10 klockan 00:36 +0800 skrev Zhao Zhili:
> > > From: Zhao Zhili <zhilizhao@tencent.com>
> > >
> > > android_get_device_api_level() is a static inline method before
> > > API
> > > level 29. It was implemented via __system_property_get(). We can
> > > do
> > > the same thing, but I don't want to mess up with
> > > __system_property_get.
> >
> > The code tests on API >= 24 not 29. Or do you mean that
> > android_get_device_api_level() is available since 24 but not used
> > until
> > 29?
>
> If android_get_device_api_level() is a global symbol instead of
> inline method, I would
> use dlsym() instead to make it works on build target api level < 24
> and device/runtime
> api level >= 24.
Perhaps a comment would be appropriate here as well then. A lot of the
NDK vs JNI vs versions of either isn't entirely obvious
/Tomas
_______________________________________________
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".
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-01-09 16:45 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-09 16:36 [FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: get api level via NDK method Zhao Zhili
2023-01-09 12:32 ` Tomas Härdin
2023-01-09 14:41 ` Zhao Zhili
2023-01-09 16:44 ` Tomas Härdin
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git