On Sun, Jun 16, 2019 at 01:13:38AM +0800, Jun Zhao wrote:
> From: Jun Zhao <barryjz...@tencent.com>
> 
> Redundant condition: '!A || B' is equivalent to '!A || (A && B)' but
> more clearly.
> 
> Signed-off-by: Jun Zhao <barryjz...@tencent.com>
> ---
>  libavfilter/af_asetnsamples.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/libavfilter/af_asetnsamples.c b/libavfilter/af_asetnsamples.c
> index a7e424f..bbc391a 100644
> --- a/libavfilter/af_asetnsamples.c
> +++ b/libavfilter/af_asetnsamples.c
> @@ -67,7 +67,7 @@ static int activate(AVFilterContext *ctx)
>          return ret;
>  
>      if (ret > 0) {
> -        if ((!s->pad || (s->pad && frame->nb_samples == s->nb_out_samples))) 
> {
> +        if (!s->pad || frame->nb_samples == s->nb_out_samples) {
>              ret = ff_filter_frame(outlink, frame);
>              if (ff_inlink_queued_samples(inlink) >= s->nb_out_samples)
>                  ff_filter_set_ready(ctx, 100);

LGTM

thx

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

"I am not trying to be anyone's saviour, I'm trying to think about the
 future and not be sad" - Elon Musk

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to