[FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

[Michael Niedermayer]

On Mon, Feb 22, 2016 at 12:20:36PM +0100, Matthieu Bouron wrote:
> From: Matthieu Bouron <matthieu.bouron at 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



[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

During times of universal deceit, telling the truth becomes a
revolutionary act. -- George Orwell
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160222/99693f4e/attachment.sig>