]> granicus.if.org Git - file/commit
FIx windows icons and cursors (Christoph Biedl)
authorChristos Zoulas <christos@zoulas.com>
Sat, 8 Mar 2014 17:26:48 +0000 (17:26 +0000)
committerChristos Zoulas <christos@zoulas.com>
Sat, 8 Mar 2014 17:26:48 +0000 (17:26 +0000)
commit51904f9df564dce31663e9ab3611dff836d99991
tree01f44863d326420896039c6b1ed73cda41ef1b73
parent82788554a37af59d81e59a79e92e65d11a690e05
FIx windows icons and cursors (Christoph Biedl)

the detection of MS-Windows icon files was buggy and incomplete. Since
non-animated cursors have a very similar format, they are added here,
too.

Some details: Using only the very first four octets for determination
has always some risk of false detection. So additionally look at
offset 9 which should always be zero, but according to Wikipedia might
also be 0xff.

In "number of colors" at offset 8, the zero value previously was used
to describe "256 colors", according to Wikipedia this isn't true, so
it was dropped.

On a side node: This introduces another usage of name/use. I'm afraid
we might sooner or later end up in name clashes on these identifiers.
Feel free to extend them like prefixing with the file name "msdos-".

Also, I was a bit surprised no "message" in "use" is printed, I wanted
to do

    >>4 use     ico-dir     MS Windows icon resource

but that message gets lost. Hence the hackery

    >>0 byte    x           MS Windows icon resource
    >>4 use     ico-dir

So a feature request: Print any message in "use".

This patch by the way fixes Debian#562250. It just got a little
bigger.
magic/Magdir/msdos