Проблемы с изображением

Некие проблемы с DIB?

Проблемы с изображением можно увидеть на

• jajc
• Русский фотограф – можно списаться с автором
• Заставке к КОМПАС – падает при запуске без патча
• Также на этом падает Консультант+ (если запускать с патчем ;))

Ситуация:
1. код около функции BITMAP_Object неверный
2. патч тоже неверный
3. в /wine/dlls/gdi/tests/bitmap.c
должны быть тесты об этом, они тоже неверные (если их исправить, они должны работать на Windows и не работать в wine)

Патч, корректирующий проблему для КОМПАСА
Index: bitmap.c =================================================================== RCS file: /home/wine/wine/dlls/gdi/bitmap.c,v retrieving revision 1.7 diff -u -u -r1.7 bitmap.c --- bitmap.c 13 Apr 2005 16:11:18 -0000 1.7 +++ dlls/gdi/bitmap.c 17 May 2005 18:44:52 -0000 @@ -612,6 +612,7 @@ /*********************************************************************** * BITMAP_GetObject + * Fill buffer with BITMAP or DIPSECTION (depends on dib field and count size) */ static INT BITMAP_GetObject( HGDIOBJ handle, void *obj, INT count, LPVOID buffer ) { @@ -621,26 +622,19 @@ { if( !buffer ) return sizeof(DIBSECTION); - if (count < sizeof(DIBSECTION)) - { - if (count > sizeof(BITMAP)) count = sizeof(BITMAP); - } - else - { - if (count > sizeof(DIBSECTION)) count = sizeof(DIBSECTION); - } - - memcpy( buffer, bmp->dib, count ); - return count; - } - else - { - if( !buffer ) - return sizeof(BITMAP); - if (count > sizeof(BITMAP)) count = sizeof(BITMAP); - memcpy( buffer, &bmp->bitmap, count ); - return count; + if (count >= sizeof(DIBSECTION)) + { + count = sizeof(DIBSECTION); + memcpy( buffer, bmp->dib, count ); + return count; + } + if (count > sizeof(BITMAP)) count = sizeof(BITMAP); } + if( !buffer ) + return sizeof(BITMAP); + if (count > sizeof(BITMAP)) count = sizeof(BITMAP); + memcpy( buffer, &bmp->bitmap, count ); + return count; }

//Похоже надо через таблицу цветов превращать из 24х битовой в 4х битовую....
При этом надо учитывать, что кроме 24х битового дисплея бывает еще и 32 и 16.... а иногда и 8ми.....
В теории можно повнимательней просмотреть код около функции X11_GetDIBits() (или как то так она называется) и еще (скорее всего) необходимо сделать схожие изменения в функции X11_SetBitmapBits...//

CyrilMargorin /10.12.2005 01:38/
Появилась для обсуждение идея, перевести функцию CopyImage с Get/SetBitmapBits на Get/SetDIBits. Попытка сделать это в общем-то была успешна....
Index: dlls/user/cursoricon.c =================================================================== RCS file: /home/wine/wine/dlls/user/cursoricon.c,v retrieving revision 1.22 diff -u -r1.22 cursoricon.c --- dlls/user/cursoricon.c 5 Dec 2005 20:07:34 -0000 1.22 +++ dlls/user/cursoricon.c 9 Dec 2005 22:29:21 -0000 @@ -2322,14 +2322,39 @@ if (!GetObjectW( hnd, sizeof(bm), &bm )) return 0; bm.bmBits = NULL; - if ((res = CreateBitmapIndirect(&bm))) - { - char *buf = HeapAlloc( GetProcessHeap(), 0, bm.bmWidthBytes * bm.bmHeight ); - GetBitmapBits( hnd, bm.bmWidthBytes * bm.bmHeight, buf ); - SetBitmapBits( res, bm.bmWidthBytes * bm.bmHeight, buf ); - HeapFree( GetProcessHeap(), 0, buf ); - } - return (HICON)res; + if (!screen_dc) screen_dc = CreateDCW( DISPLAYW, NULL, NULL, NULL ); + if (screen_dc) + { + if((res = CreateCompatibleBitmap(screen_dc,bm.bmWidth,bm.bmHeight))) + { + BITMAPINFO* pbmi; + char *buf; + char *bmibuf; + int nColor=0; + if(bm.bmBitsPixel <= 8) + nColor = 1<<bm.bmBitsPixel; + if(bm.bmBitsPixel == 15 || bm.bmBitsPixel == 16) + nColor = 3; + bmibuf = HeapAlloc( GetProcessHeap(), 0, sizeof(BITMAPINFOHEADER)+nColor*sizeof(RGBQUAD)); + pbmi = (BITMAPINFO*)bmibuf; + ZeroMemory(pbmi,sizeof(BITMAPINFOHEADER)+nColor*sizeof(RGBQUAD)); + pbmi->bmiHeader.biSize = sizeof(pbmi->bmiHeader); + pbmi->bmiHeader.biWidth = bm.bmWidth; + pbmi->bmiHeader.biHeight = bm.bmHeight; + pbmi->bmiHeader.biPlanes = bm.bmPlanes; + pbmi->bmiHeader.biBitCount = bm.bmBitsPixel; + pbmi->bmiHeader.biCompression = BI_RGB; + buf = HeapAlloc( GetProcessHeap(), 0, bm.bmWidthBytes * bm.bmHeight ); + GetDIBits(screen_dc,hnd,0,bm.bmHeight,buf,pbmi,DIB_RGB_COLORS); + SetDIBits(screen_dc,res,0,bm.bmHeight,buf,pbmi,DIB_RGB_COLORS); + HeapFree( GetProcessHeap(), 0, bmibuf ); + HeapFree( GetProcessHeap(), 0, buf ); + } + return (HICON)res; + } + /* In case of error it should return 0 + */ + return 0; } case IMAGE_ICON: return CURSORICON_ExtCopy(hnd,type, desiredx, desiredy, flags);