On 2011-12-20 09:41:49 -0500, Sean McGovern wrote: > On Tuesday, November 8, 2011, Kostya Shishkov <[email protected]> > wrote: > > On Tue, Oct 18, 2011 at 10:49:20PM +0200, Janne Grunau wrote: > >> From: Laurent Aimar <[email protected]> > >> > >> On Fri, Oct 07, 2011 at 03:26:15PM +0200, Janne Grunau wrote: > >> > From: Laurent Aimar <[email protected]> > >> > > >> > Signed-off-by: Janne Grunau <[email protected]> > >> > --- > >> > libavcodec/sunrast.c | 16 +++++++++------- > >> > 1 files changed, 9 insertions(+), 7 deletions(-) > >> > > Sorry for not ringing in on this sooner, but: > > 'type' is an unsigned int, so the first part of this statement always > evaluates to zero. > > From sunrast.c: > > 26: #define RT_OLD 0 > 53: unsigned int w, h, depth, type, maptype, maplength, stride, x, y, len, > alen; > 77: if (type < RT_OLD || type > RT_FORMAT_IFF) {
not really problem. Does anyone care enough to end a patch? I won't. Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
