On Tue, 29 May 2018, Carl Eugen Hoyos wrote:

Hi!

Attached patch makes debugging a little easier imo.

Please comment, Carl Eugen

diff --git a/fftools/ffplay.c b/fftools/ffplay.c
index dcca9c2..f9571d7 100644
--- a/fftools/ffplay.c
+++ b/fftools/ffplay.c
@@ -2578,7 +2578,7 @@ static int stream_component_open(VideoState *is, int 
stream_index)
        if (forced_codec_name) av_log(NULL, AV_LOG_WARNING,
                                      "No codec could be found with name 
'%s'\n", forced_codec_name);
        else                   av_log(NULL, AV_LOG_WARNING,
-                                      "No codec could be found with id %d\n", 
avctx->codec_id);
+                                      "No codec could be found with id %d (%s)\n", 
avctx->codec_id, avcodec_get_name(avctx->codec_id));

Maybe go one step further, and change the error message to

"No decoder could be found for codec %s\n", avcodec_get_name(avctx->codec_id)

I don't see any use for dumping the codec_id, it has no usefulness to the end user IMHO.

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to