lavfi/vf_idet: remove the loop in request_frame().
authorNicolas George <george@nsup.org>
Tue, 25 Aug 2015 18:31:50 +0000 (20:31 +0200)
committerNicolas George <george@nsup.org>
Sun, 20 Sep 2015 16:50:00 +0000 (18:50 +0200)
It is not necessary due to the use of FF_LINK_FLAG_REQUEST_LOOP.

libavfilter/vf_idet.c

index 711ba6e..d460330 100644 (file)
@@ -314,7 +314,7 @@ static int request_frame(AVFilterLink *link)
     AVFilterContext *ctx = link->src;
     IDETContext *idet = ctx->priv;
 
-    do {
+    // TODO reindent
         int ret;
 
         if (idet->eof)
@@ -328,14 +328,11 @@ static int request_frame(AVFilterLink *link)
             if (!next)
                 return AVERROR(ENOMEM);
 
-            filter_frame(link->src->inputs[0], next);
+            ret = filter_frame(link->src->inputs[0], next);
             idet->eof = 1;
-        } else if (ret < 0) {
-            return ret;
         }
-    } while (link->frame_requested);
 
-    return 0;
+    return ret;
 }
 
 static av_cold void uninit(AVFilterContext *ctx)