a better patch (I think).

On Sat, Dec 12, 2009 at 7:43 PM, patrick keshishian <pkesh...@gmail.com> wrote:
> Slight adjustment to patch.
>
> On Sat, Dec 12, 2009 at 7:24 PM, patrick keshishian <pkesh...@gmail.com> 
> wrote:
>> Hello,
>>
>> This patch lets xv load uncompressed BMP files with negative height
>> value in the header.
>>
>> $ file rightsideup.bmp
>> rightsideup.bmp: PC bitmap data, Windows 3.x format, 320 x -239 x 32
>>
>> Google pointed me to wikipedia[0] on this subject. Evidently, if the
>> height value is negative for uncompressed BMP files, it is an
>> indication that the bitmap data is stored top to bottom[1].
>>
>> Pre-patch, xv refuses to open these images saying that dimensions are
>> too large. This patch addresses this issue (the patch is
>> post-jumbo-patch applied).
>>
>> I have already forwarded this patch to Greg Roelofs who maintains the
>> xv-Jumbo patch.
>>
>> I tested xv after my patch to make sure it didn't break anything using
>> the "Test BMP images"[2] referenced from the wikipedia page[0]. All
>> work except the bottom three which xv didn't like prior to the patch.
>>
>> Unfortunately, the "rightsideup.bmp" image, and friends, I ran into
>> are copyrighted content which I do not have permission to
>> redistribute. Also, ATM I don't know what tool was used to generate
>> them.
>>
>> Comments? OK?
>>
>> Tested on macppc late November snapshot.
>>
>> Cheers,
>> --patrick
>>
>>
>> [0] http://en.wikipedia.org/wiki/BMP_file_format
>> [1] From [0]: "Uncompressed Windows bitmaps can also be stored from
>> the top row to the bottom, if the image height value is negative."
>> [2] http://wvnvaxa.wvnet.edu/vmswww/bmp.html
>>
>

Attachment: xv-patch
Description: Binary data



Reply via email to