On Mon, Feb 22, 2016 at 01:08:49PM +0100, Michael Niedermayer wrote:
> On Mon, Feb 22, 2016 at 12:20:36PM +0100, Matthieu Bouron wrote:
> > From: Matthieu Bouron <matthieu.bou...@stupeflix.com>
> [...]
> > + codec = (*env)->NewObject(env, jfields.mediacodec_list_class,
> > jfields.init_id, 0);
> > + if (!codec) {
> > + av_log(NULL, AV_LOG_ERROR, "Could not create media codec
> > list\n");
> > + goto done;
> > + }
> > +
> > + tmp = (*env)->CallObjectMethod(env, codec,
> > jfields.find_decoder_for_format_id, format);
> > + if (!tmp) {
> > + av_log(NULL, AV_LOG_ERROR, "Could not find decoder in media
> > codec list\n");
> > + goto done;
> > + }
> > +
> > + name = ff_jni_jstring_to_utf_chars(env, tmp, NULL);
> > + if (!name) {
>
> > + av_log(NULL, AV_LOG_ERROR, "Could not convert jstring to utf
> > chars\n");
>
> some non NULL context would be better, if possible, so the user knows
> where an error came from
It would require to pass the log_ctx (avctx) as an argument of
ff_AMediaCodecList_getCodecByName.
All the functions of this wrapper does not take a log_ctx as argument
to be as close as possible to the original NDK MediaCodec API. The
NDK MediaCodec API does not provide any wrapper around MediaCodecList.
I would like the whole wrapper API to be consistent but also as close as
possible to original one.
If you think it's mandatory I can add a log_ctx argument to every
functions of the wrapper API (so it will be used directly by av_log and
ff_jni_exception_check).
On another note, I fixed locally this part of the code by adding missing calls
to ff_jni_exception_check.
[...]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
