Re: gdiplus: Support for indexed formats in GdipBitmapSetPixel

2011-01-11 Thread Paul Vriens
On 01/11/2011 02:26 AM, MP wrote: +case PixelFormat8bppIndexed: +setpixel_8bppIndexed(r,g,b,a,row,x,bitmap); +break; +case PixelFormat4bppIndexed: +setpixel_8bppIndexed(r,g,b,a,row,x,bitmap); +break; +case

Re: [PATCH 1/3] winemenubuilder: Icon heights/widths of 0 mean 256, for Vista icons.

2011-01-11 Thread Damjan Jovanovic
The 256x256 pixel icons are designed to be viewed in that size. They can, and often do, contain a different picture to the smaller icons. When you scale them down to a smaller size, they look bad. On MacOS that may not matter since multiple icon sizes are written and the best is chosen, but on

Re: fusion/tests: Add some tests for IAssemblyCache_QueryAssemblyInfo.

2011-01-11 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=8223 Your paranoid

Re: sc2 performance (was: Wine and D3D)

2011-01-11 Thread Stefan Dösinger
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am 10.01.2011 um 22:03 schrieb Emanuele Oriani: 2) Stefan, do you have any hint on where we should start optimizing the calls of D3D - OpenGL functions? Where do we waste time/CPU cycles? A few bugs that are probably easier to fix(there's no

Re: po: Fix rc-po misconverted iccvid French translation

2011-01-11 Thread Alexandre Julliard
Frédéric Delanoy frederic.dela...@gmail.com writes: --- po/fr.po |8 1 files changed, 4 insertions(+), 4 deletions(-) There are many more issues to fix. One of the motivations for po files is to reduce the patch churn caused by translations, so please don't send a separate patch

Re: po: Fix rc-po misconverted iccvid French translation

2011-01-11 Thread Frédéric Delanoy
2011/1/11 Alexandre Julliard julli...@winehq.org: Frédéric Delanoy frederic.dela...@gmail.com writes: ---  po/fr.po |    8  1 files changed, 4 insertions(+), 4 deletions(-) There are many more issues to fix. One of the motivations for po files is to reduce the patch churn caused

Re: [4/7] kernel32/tests: Demonstrate an input event handling peculiarity with WriteConsoleInputW.

2011-01-11 Thread Eric Pouech
Hi Andrew, I think the test should be enhanced by checking the information about mouse position it seems that windows coalesce two mouse messages, but does it also occur when the position is different (a cursory look seems to indicate that you resend the same coordinate for mouse move) I'd expect

Re: po: Fix rc-po misconverted iccvid French translation

2011-01-11 Thread Alexandre Julliard
Frédéric Delanoy frederic.dela...@gmail.com writes: Actually I based on the Wine translation statistics page, which most probably will need a overhaul since it didn't find other non translated/correctly converted strings. BTW while reviewing the po file, I stumbled upon a potential problem:

Re: [3/7] kernel32: Improve parameter validation for ReadConsoleOutputAttribute.

2011-01-11 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=8233 Your paranoid

Re: [2/7] kernel32: Improve parameter validation for ReadConsoleOutputCharacterA.

2011-01-11 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=8232 Your paranoid

Re: [4/7] kernel32/tests: Demonstrate an input event handling peculiarity with WriteConsoleInputW.

2011-01-11 Thread Andrew Nguyen
On 01/11/2011 05:39 AM, Eric Pouech wrote: Hi Andrew, I think the test should be enhanced by checking the information about mouse position it seems that windows coalesce two mouse messages, but does it also occur when the position is different (a cursory look seems to indicate that you

Re: [2/7] kernel32: Improve parameter validation for ReadConsoleOutputCharacterA.

2011-01-11 Thread Andrew Nguyen
On 01/11/2011 05:32 AM, Marvin wrote: Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at

Re: [PATCH 4/4] comctl32/rebar: Implemented dragging between rows

2011-01-11 Thread Joris Huizer
Hello, In this patch, in function REBAR_HandleUDDrag: +if(yOff 0) +{ +/* Place the band above the current top row */ +DPA_DeletePtr(infoPtr-bands, iHitBand); +hitBand-fStyle = RBBS_BREAK; +REBAR_GetBand(infoPtr, 0)-fStyle |= RBBS_BREAK; +

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Juan Lang
Hi Christian, -static BYTE bin1[] = { 0x55, 0x53 }; -static BYTE bin2[] = { 0x4d, 0x69, 0x6e, 0x6e, 0x65, 0x73, 0x6f, 0x74, +static BYTE bin1[] = { 0x55, 0x53 }; +static BYTE bin2[] = { 0x4d, 0x69, 0x6e, 0x6e, 0x65, 0x73, 0x6f, 0x74, 0x61 }; Please don't introduce whitespace-only changes,

Re: advapi32: Only attempt multibyte conversion in SetEntriesInAclA when ptstrName is a string.

2011-01-11 Thread Alexandre Julliard
Andrew Nguyen angu...@codeweavers.com writes: --- dlls/advapi32/security.c | 19 --- 1 files changed, 12 insertions(+), 7 deletions(-) It doesn't work here: ../../../tools/runtest -q -P wine -M advapi32.dll -T ../../.. -p advapi32_test.exe.so security.c touch security.ok

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Christian Inci
Am 2011-01-11 16:18, schrieb Juan Lang: Please don't introduce whitespace-only changes, it's harder to see what your patch is doing. Sorry, that wasn't intended. Since these changes pass on unpatched wine, they don't demonstrate the necessity for your patch. We need tests that show what the

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Juan Lang
I don't know, what you've doing, but for me it show errors. (on unpatched crypt32/str.c (and patched crypt32/tests/str.c ;) )) Ah, right. Sorry, I wasn't thinking clearly. In that case, you need to number the patches as a series to show the dependency between them. --Juan

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Christian Inci
Am 2011-01-11 16:58, schrieb Juan Lang: Ah, right. Sorry, I wasn't thinking clearly. In that case, you need to number the patches as a series to show the dependency between them. --Juan First the normal then the test, or reverse? --Christian

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Juan Lang
First the normal then the test, or reverse? Tests must pass with every commit. In general, I prefer to see tests first, with failing tests marked todo_wine. Then the second patch, with implementation, removes the todo_wines. In this case adding todo_wine might be more trouble than it's worth,

Re: crypt32: Added Quoting-Tests

2011-01-11 Thread Christian Inci
Am 2011-01-11 17:03, schrieb Juan Lang: Tests must pass with every commit. In general, I prefer to see tests first, with failing tests marked todo_wine. Then the second patch, with implementation, removes the todo_wines. In this case adding todo_wine might be more trouble than it's worth,

re: crypt32: Fix Quote-Handling (try 2)

2011-01-11 Thread Juan Lang
Hi Christian, on second thought I don't have too much to say about this, other than style comments: take them or leave them. +/* Only use this function internally. */ This comment isn't all that useful. You might indicate that quotes are only needed by CertNameToStr when the string type is

Re: crypt32: Fix Quote-Handling (try 2)

2011-01-11 Thread Christian Inci
Am 2011-01-11 17:11, schrieb Juan Lang: Also, since CertRDNValueToStr doesn't need quote_string, you should probably invert the order in which they appear in the file. The calls or the definitions? If the definition, should I move is_quotable_char too? --Christian

Re: crypt32: Fix Quote-Handling (try 2)

2011-01-11 Thread Juan Lang
The calls or the definitions? If the definition, should I move is_quotable_char too? The definitions. Yes, please. --Juan

AppDB: Do not upload screenshots of error messages, installers, game menus

2011-01-11 Thread Joerg-Cyril.Hoehle
Hi, the AppDB screenshot submission pages mentions this rule - Do not upload screenshots of error messages, installers, game menus yet it is continuously violated and e.g. screenshots of game menus are accepted, including recently. Is it still a useful rule? If yes, should matching screenshots

Re: AppDB: Do not upload screenshots of error messages, installers, game menus

2011-01-11 Thread Rosanne DiMesio
On Tue, 11 Jan 2011 18:07:48 +0100 joerg-cyril.hoe...@t-systems.com wrote: Hi, the AppDB screenshot submission pages mentions this rule - Do not upload screenshots of error messages, installers, game menus yet it is continuously violated and e.g. screenshots of game menus are accepted,

Re: AppDB: Do not upload screenshots of error messages, installers, game menus

2011-01-11 Thread André Hentschel
Am 11.01.2011 18:54, schrieb Rosanne DiMesio: On Tue, 11 Jan 2011 18:07:48 +0100 joerg-cyril.hoe...@t-systems.com wrote: Hi, the AppDB screenshot submission pages mentions this rule - Do not upload screenshots of error messages, installers, game menus yet it is continuously violated and

Procedure for adding message contexts to PO messages?

2011-01-11 Thread Frédéric Delanoy
Hi, I'm quite green wrt PO files and was reviewing French PO file after recent rc - po migration. So I read somewhat about PO format and Wine's PO support code. I wonder how to add message contexts, and thought it might be this way: 1. Add msgctxt myContext next to the associated msgid myMsgid

Re: AppDB: Do not upload screenshots of error messages, installers, game menus

2011-01-11 Thread Rosanne DiMesio
On Tue, 11 Jan 2011 19:18:17 +0100 André Hentschel n...@dawncrow.de wrote: AppDB should represent the actual state of an App, so game menus, installer screenshots and graphical glitches are ok IMO. Of course the graphical glitch should have a bug filed and should not only lay around in

wineserver hitting kernel bug?

2011-01-11 Thread André Hentschel
Hi all, i get err:process:start_wineboot failed to start wineboot, err 1359 which of course leads to major problems, i tracked it down to server/process.c in function process_poll_event it hits the line if (event (POLLERR | POLLHUP)) kill_process( process, 0 ) because event is 0x19 what can

Re: wineserver hitting kernel bug?

2011-01-11 Thread Austin English
2011/1/11 André Hentschel n...@dawncrow.de: Hi all, i get  err:process:start_wineboot failed to start wineboot, err 1359 which of course leads to major problems, i tracked it down to server/process.c in function process_poll_event it hits the line  if (event (POLLERR | POLLHUP))

Re: wineserver hitting kernel bug?

2011-01-11 Thread Alexandre Julliard
André Hentschel n...@dawncrow.de writes: Hi all, i get err:process:start_wineboot failed to start wineboot, err 1359 which of course leads to major problems, i tracked it down to server/process.c in function process_poll_event it hits the line if (event (POLLERR | POLLHUP))

Re: sc2 performance (was: Wine and D3D)

2011-01-11 Thread Emanuele Oriani
Hi Stefan, thanks very much. Do we think any of the below points will be prioritized by wine devs? Btw, I don't have windows (moral choice) so can't test it (I have a virtualized winxp but can't run the game in VirtualBox). Cheers, On 11/01/11 10:38, Stefan Dösinger wrote: -BEGIN PGP

Re: windowscodecs/tests: COM cleanup for converter.c.

2011-01-11 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=8249 Your paranoid

Re: wineserver hitting kernel bug?

2011-01-11 Thread Austin English
2011/1/11 Austin English austinengl...@gmail.com: 2011/1/11 André Hentschel n...@dawncrow.de: Hi all, i get  err:process:start_wineboot failed to start wineboot, err 1359 which of course leads to major problems, i tracked it down to server/process.c in function process_poll_event it hits

Re: msvcrt: Partially implement _set_abort_behavior

2011-01-11 Thread Detlef Riekenberg
On Mi, 2011-01-12 at 00:48 +0300, Nikolay Sivov wrote: Date: Wed, 12 Jan 2011 00:46:26 +0300 Subject: [PATCH 1/1] Partially implement _set_abort_behavior Please search the patches queue first. try 2: http://source.winehq.org/patches/data/70231 try 1: http://source.winehq.org/patches/data/70136

Re: [PATCH 1/3] winemenubuilder: Icon heights/widths of 0 mean 256, for Vista icons.

2011-01-11 Thread Ken Thomases
On Jan 11, 2011, at 2:41 AM, Damjan Jovanovic wrote: The 256x256 pixel icons are designed to be viewed in that size. They can, and often do, contain a different picture to the smaller icons. When you scale them down to a smaller size, they look bad. On MacOS that may not matter since multiple