Re: [AD] Possible bug in 4.2 bitmap loading code.

["Peter Hull" <peterhull90@xxxxxxxxxx>]

Is there any plan to add this to 4.2? I ask because I've just noticed
bmp.c still showing as 'locally modified' in my SVN directory.

Pete


On Sat, Apr 5, 2008 at 10:36 PM, Peter Wang <novalazy@xxxxxxxxxx> wrote:
> On 2008-03-23, David Capello <davidcapello@xxxxxxxxxx> wrote:
>> On Sun, Mar 23, 2008 at 9:28 PM, Peter Wang <novalazy@xxxxxxxxxx> wrote:
>> >  I am not sure why biHeight is left as an `unsigned long'.  Shouldn't it just be
>> >  changed to `signed long' and then we can get rid of the int casts?
>>
>> OK, I put it as a 'signed long' in BITMAPINFOHEADER and WINBMPINFOHEADER
>> (OS/2 can't be negative, so it stays as an 'unsigned short').
>>
>> Attached the new patch that fix problems to load
>> bitmaps from the bmp-suite of Jason Summers.
>>
>>  http://www.dacap.com.ar/bmpbug2.zip
>
> Thank you for the patch. It will be committed to 4.3.10plus and 4.9.
> Sorry for the delay.
>
>> There are another issue, MSDN says that an BMP can't be
>> 'top-down' and 'compressed' at the same time, but with Adobe
>> Photoshop you can create them (RLE4 or RLE8, with negative
>> height). ¿We should give support to load these top-down
>> compressed files? I think that most programs don't support
>> these odd files.
>
> What do we do now?
>
> Peter
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
> Register now and save $200. Hurry, offer ends at 11:59 p.m.,
> Monday, April 7! Use priority code J8TLD2.
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
> --
> https://lists.sourceforge.net/lists/listinfo/alleg-developers
>