Re: [FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Ganesh Ajjanagadde
On Sun, Aug 23, 2015 at 5:41 PM, Clément Bœsch wrote: > On Sun, Aug 23, 2015 at 05:33:11PM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Aug 23, 2015 at 5:28 PM, Clément Bœsch wrote: >> > On Sun, Aug 23, 2015 at 11:23:54PM +0200, Clément Bœsch wrote: >> >> On Sun, Aug 23, 2015 at 11:58:23AM -0400,

Re: [FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Clément Bœsch
On Sun, Aug 23, 2015 at 05:33:11PM -0400, Ganesh Ajjanagadde wrote: > On Sun, Aug 23, 2015 at 5:28 PM, Clément Bœsch wrote: > > On Sun, Aug 23, 2015 at 11:23:54PM +0200, Clément Bœsch wrote: > >> On Sun, Aug 23, 2015 at 11:58:23AM -0400, Ganesh Ajjanagadde wrote: > >> [...] > >> > diff --git a/lib

Re: [FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Ganesh Ajjanagadde
On Sun, Aug 23, 2015 at 5:28 PM, Clément Bœsch wrote: > On Sun, Aug 23, 2015 at 11:23:54PM +0200, Clément Bœsch wrote: >> On Sun, Aug 23, 2015 at 11:58:23AM -0400, Ganesh Ajjanagadde wrote: >> [...] >> > diff --git a/libavfilter/vf_hqx.c b/libavfilter/vf_hqx.c >> > index fa15d9c..0178793 100644 >>

Re: [FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Clément Bœsch
On Sun, Aug 23, 2015 at 11:23:54PM +0200, Clément Bœsch wrote: > On Sun, Aug 23, 2015 at 11:58:23AM -0400, Ganesh Ajjanagadde wrote: > [...] > > diff --git a/libavfilter/vf_hqx.c b/libavfilter/vf_hqx.c > > index fa15d9c..0178793 100644 > > --- a/libavfilter/vf_hqx.c > > +++ b/libavfilter/vf_hqx.c >

Re: [FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Clément Bœsch
On Sun, Aug 23, 2015 at 11:58:23AM -0400, Ganesh Ajjanagadde wrote: [...] > diff --git a/libavfilter/vf_hqx.c b/libavfilter/vf_hqx.c > index fa15d9c..0178793 100644 > --- a/libavfilter/vf_hqx.c > +++ b/libavfilter/vf_hqx.c > @@ -65,9 +65,9 @@ static av_always_inline int yuv_diff(uint32_t yuv1, > u

[FFmpeg-devel] [PATCH 2/2] all: do standards compliant absdiff computation

2015-08-23 Thread Ganesh Ajjanagadde
This resolves undefined behavior, and also silences -Wabsolute-value in clang 3.5+. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/flashsv2enc.c | 6 +++--- libavfilter/vf_hqx.c | 6 +++--- libavfilter/vf_xbr.c | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a