February 2008 - Wine-Devel - list.winehq.org

Re: user32dll: cursoricon.c. Fix 16b BMP handling with BT_BITFIELD.

by Dmitry Timoshkov

"Alexander Demakin" <avd(a)etersoft.ru> wrote: > This patch fixes image header handling with BT_BITFIELD flag. 12 bytes of mask > were included into image bits, it caused some color troubles and image > horizontal shift. Size of bitmap info structure includes now this bytes. You are mixing tabs and spaces again, and please use 4 spaces for indentation. > static int bitmap_info_size( const BITMAPINFO * info, WORD coloruse ) > { > int colors; > - > + int result; > if (info->bmiHeader.biSize == sizeof(BITMAPCOREHEADER)) Any reason you have removed a space after a variable declaration? > Changelog: > user32.dll Fix 16b BMP handling with BT_BITFIELD. Please have a look how others name the patches. A usual convenience is to write dll name without extension, ': ' and short description. -- Dmitry.

Feb. 20, 2008

Re: gdi32dll: bitblt.c. Fix copying from uninitialized memory

by Dmitry Timoshkov

"Alexander Demakin" <avd(a)etersoft.ru> wrote: > This patch adds check of return value of GetDIBits in BitBlt.c. If a error > occurs, GetDIBits returns 0 and bits variable points to uninitialized data. > It always occurs when calling BitBlt for printer context due to > implementation of PSDRV_SelectBitmap(). > Changelog: > gdi32.dll Fix copying from uninitialized memory. You are mixing tabs and spaces in the patch, otherwise tha patch looks good to me. -- Dmitry.

Feb. 20, 2008

Re: dlls/msi/streams.c type fixes

by Gerald Pfeifer

On Sun, 17 Feb 2008, James Hawkins wrote: >> static INT add_streams_to_table(MSISTREAMSVIEW *sv) >> + /* Return -1 in case of error. */ > Don't add random comments like this. Okay, will keep this in mind, especially for dlls/msi. Gerald

Feb. 20, 2008

Re: [1/2] user32:scroll: Added some tests for scrollbar [try2]

by Anatoly Lyutin

Alexandre Julliard wrote: > The tests fail: > > ../../../tools/runtest -q -P wine -M user32.dll -T ../../.. -p > user32_test.exe.so scroll.c && touch scroll.ok > scroll.c:56: Test failed: The scrollbar window should be disabled. > scroll.c:122: Test failed: The ShowScrollBar() should failed. > Yes! These tests must be failed in WINE, but it passes clear in Win XP. I just want to show you some incorrect behavior of scrollbar in WINE. Should I mark these tests as "todo_wine"? -- Best regards Anatoly Lyutin.

Feb. 20, 2008

Re: add rule for the .m suffix

by Emmanuel Maillard

Hi, Did you convince Alexandre Julliard to include Objective-C on wine tree ?? Because you need to if you want to submit my patches to winehq, or rewrite it in plain C with Carbon. Emmanuel Le 20 févr. 08 à 13:31, Steven Edwards a écrit : > This is needed for quartzdrv patch. Trying to reduce the size of the > diff, I expect a real winequartzdrv will need it later on anyway > > -- > Steven Edwards > > "There is one thing stronger than all the armies in the world, and > that is an idea whose time has come." - Victor Hugo > <support-m-suffix.patch>

Feb. 20, 2008

Re: shell32: Support getting the shell folder path for users other than the current user and the default user (take 2)

by Alexandre Julliard

"Ge van Geldorp" <ge(a)thinstall.com> writes: > else > { > + LPWSTR sidStr = NULL; > LPCWSTR userPrefix = NULL; > HKEY hRootKey; This is not used anywhere. -- Alexandre Julliard julliard(a)winehq.org

Feb. 20, 2008

Adding devices to registry with the help of hal

by Christian Gmeiner

Hi all, I want you to present my current proof-of-conecpt. It shows how we can use hal.c of explorer to create registry keys for devices. Keep in mind that it only creates one registry key for each usb device at the moment. Before I investigate more time in this, I want to get some feedback about my method. If you are happy with it, I will finish this suftt :) Greets, Christian

Feb. 20, 2008

Re: [advapi32] QueryServiceConfig2 (3rd try) [resend]

by Alexandre Julliard

Stefan Leichter <Stefan.Leichter(a)camline.com> writes: > + switch(dwLevel) { > + case SERVICE_CONFIG_DESCRIPTION: { > + static const WCHAR szDescription[] = {'D','e','s','c','r','i','p','t','i','o','n',0}; > + LPSERVICE_DESCRIPTIONW config = (LPSERVICE_DESCRIPTIONW) buffer; > + *needed = sizeof (SERVICE_DESCRIPTIONW); > + sz = size - *needed; > + r = RegQueryValueExW( hKey, szDescription, 0, &type, (LPBYTE) (config ? config + 1: NULL), &sz ); > + if((r == ERROR_SUCCESS) && ( type != REG_SZ)) { > + FIXME("SERVICE_CONFIG_DESCRIPTION: don't know how to handle type %d\n", type); > + return FALSE; > + } > + *needed += sz; > + if(config) { > + if(r == ERROR_SUCCESS) > + config->lpDescription = (LPWSTR) (config + 1); > + else > + config->lpDescription = NULL; You are still not handling overflows correctly. What happens if size is less than sizeof(SERVICE_DESCRIPTIONW)? -- Alexandre Julliard julliard(a)winehq.org

Feb. 20, 2008

Re: [1/2] user32:scroll: Added some tests for scrollbar [try2]

by Alexandre Julliard

Anatoly Lyutin <vostok(a)etersoft.ru> writes: > Changelog: > Added some tests for scrollbar The tests fail: ../../../tools/runtest -q -P wine -M user32.dll -T ../../.. -p user32_test.exe.so scroll.c && touch scroll.ok scroll.c:56: Test failed: The scrollbar window should be disabled. scroll.c:122: Test failed: The ShowScrollBar() should failed. make: *** [scroll.ok] Error 2 -- Alexandre Julliard julliard(a)winehq.org

Feb. 20, 2008

Adding devices to registry with the help of hal

by Christian Gmeiner

Hi all, I want you to present my current proof-of-conecpt. It shows how we can use hal.c of explorer to create registry keys for devices. Keep in mind that it only creates one registry key for each usb device at the moment. Before I investigate more time in this, I want to get some feedback about my method. If you are happy with it, I will finish this suftt :) Greets, Christian

Feb. 20, 2008