Re: [FFmpeg-devel] [PATCH 1/2] sws/yuv2rgb: avoid a few ub on signed left shifts

2016-02-27 Thread Clément Bœsch
On Sat, Feb 27, 2016 at 01:05:35AM +0100, Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 10:34:45PM +0100, Clément Bœsch wrote: > > --- > > libswscale/yuv2rgb.c | 20 ++-- > > 1 file changed, 10 insertions(+), 10 deletions(-) > > LGTM > > thanks > applied -- Clément B.

Re: [FFmpeg-devel] [PATCH 1/2] sws/yuv2rgb: avoid a few ub on signed left shifts

2016-02-26 Thread Michael Niedermayer
On Fri, Feb 26, 2016 at 10:34:45PM +0100, Clément Bœsch wrote: > --- > libswscale/yuv2rgb.c | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I do not agree with what you

[FFmpeg-devel] [PATCH 1/2] sws/yuv2rgb: avoid a few ub on signed left shifts

2016-02-26 Thread Clément Bœsch
--- libswscale/yuv2rgb.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index 3671fe3..9d79d79 100644 --- a/libswscale/yuv2rgb.c +++ b/libswscale/yuv2rgb.c @@ -824,18 +824,18 @@ av_cold int