[FFmpeg-devel] [PATCH 06/15] lavfi/vf_fps: remove looping on request_frame().

Michael Niedermayer michaelni at gmx.at
Fri Oct 2 22:57:08 CEST 2015


On Fri, Oct 02, 2015 at 05:13:17PM +0200, Nicolas George wrote:
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>  libavfilter/vf_fps.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavfilter/vf_fps.c b/libavfilter/vf_fps.c
> index 6154f6d..0f0ed24 100644
> --- a/libavfilter/vf_fps.c
> +++ b/libavfilter/vf_fps.c
> @@ -129,8 +129,7 @@ static int request_frame(AVFilterLink *outlink)
>      int frames_out = s->frames_out;

>      int ret = 0;

init is unneeded after this


>  
> -    while (ret >= 0 && s->frames_out == frames_out)
> -        ret = ff_request_frame(ctx->inputs[0]);
> +    ret = ff_request_frame(ctx->inputs[0]);

frames_out becomes unused

no more comments from me

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151002/5c08feec/attachment.sig>


More information about the ffmpeg-devel mailing list