On Tuesday, December 20, 2011, Janne Grunau <[email protected]> wrote:
> 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

I'll send something later then.

-- Sean McG.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to