Re: [FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Timothy Gu
On Tue, Aug 2, 2016 at 8:42 PM Timothy Gu wrote: > On Tue, Aug 2, 2016 at 6:30 PM James Almer wrote: > >> On 8/2/2016 7:28 PM, Timothy Gu wrote: >> > --- >> > libavcodec/jni.c| 9 - >> > libavcodec/mediacodec.c | 6 -- >> > 2 files

Re: [FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Timothy Gu
On Tue, Aug 2, 2016 at 6:30 PM James Almer wrote: > On 8/2/2016 7:28 PM, Timothy Gu wrote: > > --- > > libavcodec/jni.c| 9 - > > libavcodec/mediacodec.c | 6 -- > > 2 files changed, 8 insertions(+), 7 deletions(-) > > > > I'd say split this in two

Re: [FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread James Almer
On 8/2/2016 7:28 PM, Timothy Gu wrote: > --- > libavcodec/jni.c| 9 - > libavcodec/mediacodec.c | 6 -- > 2 files changed, 8 insertions(+), 7 deletions(-) > I'd say split this in two patches, so you can easily cherry-pick the jni part for release branches 3.0 and 3.1.

Re: [FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 03:28:10PM -0700, Timothy Gu wrote: > --- > libavcodec/jni.c| 9 - > libavcodec/mediacodec.c | 6 -- > 2 files changed, 8 insertions(+), 7 deletions(-) builds on all systems that are easy testable here [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Timothy Gu
--- libavcodec/jni.c| 9 - libavcodec/mediacodec.c | 6 -- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/libavcodec/jni.c b/libavcodec/jni.c index 32456f5..85dcf2a 100644 --- a/libavcodec/jni.c +++ b/libavcodec/jni.c @@ -20,19 +20,18 @@ * Foundation, Inc.,

Re: [FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 01:38:40PM -0700, Timothy Gu wrote: > --- > libavcodec/jni.c| 2 +- > libavcodec/mediacodec.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/jni.c b/libavcodec/jni.c > index 32456f5..e08a185 100644 > --- a/libavcodec/jni.c

[FFmpeg-devel] [PATCH] jni, mediacodec: Return ENOSYS on unsupported platforms

2016-08-02 Thread Timothy Gu
--- libavcodec/jni.c| 2 +- libavcodec/mediacodec.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/jni.c b/libavcodec/jni.c index 32456f5..e08a185 100644 --- a/libavcodec/jni.c +++ b/libavcodec/jni.c @@ -69,7 +69,7 @@ void *av_jni_get_java_vm(void