On Mon, Oct 28, 2013 at 9:19 AM, Vittorio Giovara <[email protected]> wrote: > On Mon, Oct 28, 2013 at 7:49 AM, Anton Khirnov <[email protected]> wrote: >> >> On Mon, 28 Oct 2013 03:00:39 +0100, Vittorio Giovara >> <[email protected]> wrote: >>> --- >>> libavfilter/af_asyncts.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/libavfilter/af_asyncts.c b/libavfilter/af_asyncts.c >>> index 0ddd8b6..3ecd4ff 100644 >>> --- a/libavfilter/af_asyncts.c >>> +++ b/libavfilter/af_asyncts.c >>> @@ -149,7 +149,7 @@ static int request_frame(AVFilterLink *link) >>> if (s->first_pts != AV_NOPTS_VALUE) >>> handle_trimming(ctx); >>> >>> - if (nb_samples = get_delay(s)) { >>> + if ((nb_samples = get_delay(s))) { >> >> Err...and what exactly does this fix? > > A clang warning claiming that assignment is used in single parenthesis. > In practice every other check in the sources has double (( )) around.
You could make it less ugly by adding a > 0 as well, at least to me it looks less weird then. - Hendrik _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
