[FFmpeg-cvslog] Revert "Merge commit '8b830ee9a26d47b138f12a82085cdb372f407f1e'" (avconv: Do not try to configure filter outputs without streams)

[Michael Niedermayer]

ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Sun Oct 11 12:10:42 2015 +0200| [14573b9b7cb0683310a91d5bd2e73bd575af3f49] | committer: Michael Niedermayer

Revert "Merge commit '8b830ee9a26d47b138f12a82085cdb372f407f1e'" (avconv: Do not try to configure filter outputs without streams)

FFmpeg already tests for this case in configure_output_filter() and printed a
clearer error message

example:
./ffmpeg -f lavfi -i color -f lavfi -i color -filter_complex "[1]null[x],[0][1]overlay" -f null -
before the merge / after the revert:
Filter null has a unconnected output

after the merge / before the revert:
Output pad "default" with type video of the filter instance "Parsed_null_0" of null not connected to any destination
Error configuring complex filters.
Invalid argument

This reverts commit 3e3779cd517e4d2d1f21d5eb6afdb428cd3c1aa0, reversing
changes made to 0b28039a44b33753d77116691d680bd60f3bac1f.

Reviewed-by: Ganesh Ajjanagadde <gajjanag at mit.edu>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=14573b9b7cb0683310a91d5bd2e73bd575af3f49
---

 ffmpeg_filter.c |    8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/ffmpeg_filter.c b/ffmpeg_filter.c
index 9426a88..10b2f5f 100644
--- a/ffmpeg_filter.c
+++ b/ffmpeg_filter.c
@@ -1035,12 +1035,8 @@ int configure_filtergraph(FilterGraph *fg)
         }
     avfilter_inout_free(&inputs);
 
-    for (cur = outputs, i = 0; cur; cur = cur->next, i++) {
-        OutputFilter *ofilter = fg->outputs[i];
-        if (ofilter->ost)
-            configure_output_filter(fg, ofilter, cur);
-    }
-
+    for (cur = outputs, i = 0; cur; cur = cur->next, i++)
+        configure_output_filter(fg, fg->outputs[i], cur);
     avfilter_inout_free(&outputs);
 
     if ((ret = avfilter_graph_config(fg->graph, NULL)) < 0)