avconv: move the avcodec_find_decoder() call to add_input_streams().
authorAnton Khirnov <anton@khirnov.net>
Mon, 22 Aug 2011 10:48:06 +0000 (12:48 +0200)
committerAnton Khirnov <anton@khirnov.net>
Tue, 23 Aug 2011 14:50:01 +0000 (16:50 +0200)
This makes the code simpler to understand, though it results in an
unnecessary call to avcodec_find_decoder() if the stream won't be
decoded.

avconv.c

index fc0a311..3bad6e0 100644 (file)
--- a/avconv.c
+++ b/avconv.c
@@ -1825,8 +1825,6 @@ static int init_input_stream(int ist_index, OutputStream *output_streams, int nb
     InputStream *ist = &input_streams[ist_index];
     if (ist->decoding_needed) {
         AVCodec *codec = ist->dec;
-        if (!codec)
-            codec = avcodec_find_decoder(ist->st->codec->codec_id);
         if (!codec) {
             snprintf(error, sizeof(error), "Decoder (codec id %d) not found for input stream #%d.%d",
                     ist->st->codec->codec_id, ist->file_index, ist->st->index);
@@ -2895,6 +2893,8 @@ static void add_input_streams(AVFormatContext *ic)
             ist->ts_scale = strtod(scale, NULL);
 
         ist->dec = choose_codec(ic, st, dec->codec_type, codec_names);
+        if (!ist->dec)
+            ist->dec = avcodec_find_decoder(dec->codec_id);
 
         switch (dec->codec_type) {
         case AVMEDIA_TYPE_AUDIO: