Browse code

lavfi/vf_idet: remove the loop in request_frame().

It is not necessary due to the use of FF_LINK_FLAG_REQUEST_LOOP.

Nicolas George authored on 2015/08/26 03:31:50
Showing 1 changed files
... ...
@@ -314,7 +314,7 @@ static int request_frame(AVFilterLink *link)
314 314
     AVFilterContext *ctx = link->src;
315 315
     IDETContext *idet = ctx->priv;
316 316
 
317
-    do {
317
+    // TODO reindent
318 318
         int ret;
319 319
 
320 320
         if (idet->eof)
... ...
@@ -328,14 +328,11 @@ static int request_frame(AVFilterLink *link)
328 328
             if (!next)
329 329
                 return AVERROR(ENOMEM);
330 330
 
331
-            filter_frame(link->src->inputs[0], next);
331
+            ret = filter_frame(link->src->inputs[0], next);
332 332
             idet->eof = 1;
333
-        } else if (ret < 0) {
334
-            return ret;
335 333
         }
336
-    } while (link->frame_requested);
337 334
 
338
-    return 0;
335
+    return ret;
339 336
 }
340 337
 
341 338
 static av_cold void uninit(AVFilterContext *ctx)