On Fri, Nov 13, 2015 at 9:50 AM, Ganesh Ajjanagadde <gajja...@mit.edu> wrote: > On Wed, Oct 28, 2015 at 10:19 PM, Michael Niedermayer > <mich...@niedermayer.cc> wrote: >> On Tue, Oct 27, 2015 at 07:58:03PM -0400, Ganesh Ajjanagadde wrote: >>> In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely >>> typo originally. >>> >>> Found by enabling -Wparentheses on clang 3.7 and running a manual audit. >>> To the best of my knowledge, no such instances remain. >>> >>> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> >>> --- >>> libavformat/rtpdec_mpa_robust.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/libavformat/rtpdec_mpa_robust.c >>> b/libavformat/rtpdec_mpa_robust.c >>> index 6442a42..91ee98a 100644 >>> --- a/libavformat/rtpdec_mpa_robust.c >>> +++ b/libavformat/rtpdec_mpa_robust.c >>> @@ -166,7 +166,7 @@ static int mpa_robust_parse_packet(AVFormatContext >>> *ctx, PayloadContext *data, >>> "Received packet without a start fragment; dropping.\n"); >>> return AVERROR(EAGAIN); >>> } >>> - if (adu_size = data->adu_size || >>> + if (adu_size == data->adu_size || >> >> i suspect that this should be != >> >> Gilles, can you confirm ? > > Been over 2 weaks, is it ok to push with the !=?
Is minor, no response, so pushed with the !=. Thanks. > >> >> [...] >> -- >> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB >> >> It is what and why we do it that matters, not just one of them. >> >> _______________________________________________ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
