On 27 November 2013 17:11, Vittorio Giovara <[email protected]> wrote: > From: Michael Niedermayer <[email protected]> > > --- > No sample to check the fix for this case too. > Vittorio > > libavcodec/dsputil.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c > index fbdd5ad..65f72b1 100644 > --- a/libavcodec/dsputil.c > +++ b/libavcodec/dsputil.c > @@ -1742,7 +1742,7 @@ void ff_set_cmp(DSPContext* c, me_cmp_func *cmp, int > type){ > > static void add_bytes_c(uint8_t *dst, uint8_t *src, int w){ > long i; > - for(i=0; i<=w-sizeof(long); i+=sizeof(long)){ > + for(i=0; i<=w-(int)sizeof(long); i+=sizeof(long)){ > long a = *(long*)(src+i); > long b = *(long*)(dst+i); > *(long*)(dst+i) = ((a&pb_7f) + (b&pb_7f)) ^ ((a^b)&pb_80); > @@ -1767,7 +1767,7 @@ static void diff_bytes_c(uint8_t *dst, uint8_t *src1, > uint8_t *src2, int w){ > } > }else > #endif > - for(i=0; i<=w-sizeof(long); i+=sizeof(long)){ > + for(i=0; i<=w-(int)sizeof(long); i+=sizeof(long)){ > long a = *(long*)(src1+i); > long b = *(long*)(src2+i); > *(long*)(dst+i) = ((a|pb_80) - (b&pb_7f)) ^ ((a^b^pb_80)&pb_80); > --
looks like it should be squashed with the first patch (and the same review applies) _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
