[WIN32K]
authorTimo Kreuzer <timo.kreuzer@reactos.org>
Sun, 6 Jun 2010 22:01:41 +0000 (22:01 +0000)
committerTimo Kreuzer <timo.kreuzer@reactos.org>
Sun, 6 Jun 2010 22:01:41 +0000 (22:01 +0000)
commita3214996fc9a71a537779b3b732aeb9aecc5dddc
tree682f7c0f792abe9070ad4bde2d58e699ab09c695
parent9a05924c626d1b152694b95f6bc96316719b6666
[WIN32K]
Rewrite the bitmap API. There were a lot of bugs. NtGdiCreateBitmap allowed a negative height, leading to either topdown or bottomup bitmaps, a behaviour that Windows doesn't have. The function copied the bitmap bits directly from the caller to the bitmap using RtlCopyMemory, ignoring different scanline length and direction (resulting in bitmaps being upside down), not SEH protected. This function (IntSetBitmapBits) is replaced by a better solution UnsafeSetBitmapBits, that takes these things into account. The name is chosen to give a hint that the function can/should be SEH protected. IntSetBitmapBits is still there, as its retarded behaviour is actually required in some places. There were also IntCreateBitmap and IntGdiCreateBitmap, now both being replaced by GreCreateBitmap. The code that set the palette is removed, as it's already done in SURFACE_AllocSurface, here gpalRGB is replaced with gpalBGR, fixing some inverted color issues. The alignment correction in SURFACE_bSetBitmapBits is reapplied, now that the callers are behaving as they are supposed to do.

svn path=/branches/reactos-yarotows/; revision=47641
subsystems/win32/win32k/eng/surface.c
subsystems/win32/win32k/include/bitmaps.h
subsystems/win32/win32k/include/surface.h
subsystems/win32/win32k/objects/bitmaps.c
subsystems/win32/win32k/objects/brush.c
subsystems/win32/win32k/objects/dibobj.c
subsystems/win32/win32k/objects/pen.c
subsystems/win32/win32k/objects/stockobj.c