September 2009 - Wine-Devel

Re: user32: Add check for the need to redraw static control after STM_SETIMAGE and STM_SETICON
by Nikolay Sivov Sept. 28, 2009

Sept. 28, 2009

Ilya Shpigor wrote: > Hi, > > This patch fix the redrawing static controls after the processing STM_SETIMAGE > and STM_SETICON messages with errors. > --- + if ((style & SS_TYPEMASK) != SS_ICON) return (HICON)-1; ... + if ((style & SS_TYPEMASK) != SS_BITMAP) return (HBITMAP)-1; ... + if ((style & SS_TYPEMASK) != SS_ENHMETAFILE) return (HENHMETAFILE)-1; --- Add tests for that please (MSDN tells about 0 return value on error).

3 3

Re: comctl32/tests: Fix monthcal test on NT4
by Nikolay Sivov Sept. 28, 2009

Sept. 28, 2009

Ge van Geldorp wrote: > Ignore MCHT_TITLEBK, title elements might be adjacent without any > background in between. > > Changelog: > comctl32/tests: Fix monthcal test on NT4 > > --- > dlls/comctl32/tests/monthcal.c | 17 +++++++++-------- > 1 files changed, 9 insertions(+), 8 deletions(-) > Hi, Ge. This one interfere with a mine, sent yesterday: http://www.winehq.org/pipermail/wine-patches/2009-September/078955.html Could you wait a bit? I hope this could fix some failures.

1 0

Running 13 months locales
by Nikolay Sivov Sept. 27, 2009

Sept. 27, 2009

Hi. Working on monthcal control I've realized that some locales could have 13 months calendars. How could I guess that I'm running such locale? After a brief look I didn't find anything useful in GetLocaleInfo().

3 5

Re: newbie question
by James Mckenzie Sept. 27, 2009

Sept. 27, 2009

>Hello All, > >can somebody tell weather it is possible to run NT service on Wine, >especially written in Delphi VCL(TService) > Services do not natively run on Wine startup. You have to start them and then leave at least one window open. If your service depends on direct hardware access or the use of any special device driver, it will not work in Wine as it does not recreate all of the functionality of Windows. You are welcome to attempt to try and run your service under Wine and advise what happened. James McKenzie

3 2

New version of UUID lookup tool
by Mikołaj Zalewski Sept. 27, 2009

Sept. 27, 2009

Hi, I had little time to work on Wine lately, but recently I could work a bit on my UUID database. The new version retrieves the identifier name after __declspec(uuid) macros and is available as an online app on AppEngine at http://mikolajapp.appspot.com/uuid/ . The offline version with the source code is available at http://www.mikolaj.zalewski.pl/dl/uuid2.tgz . Mikołaj

1 0

Re: programs: add extrac32 tool
by Nikolay Sivov Sept. 27, 2009

Sept. 27, 2009

Andrey Turkin wrote: > > Taken from Etersoft git tree with some small fixes (code style, > constness etc) and minimal /C mode support required to fix bug 18097. > --- > configure.ac | 1 + > programs/extrac32/Makefile.in | 16 ++ > programs/extrac32/extrac32.c | 423 > +++++++++++++++++++++++++++++++++++++++++ > programs/extrac32/parsecmd.c | 100 ++++++++++ > 4 files changed, 540 insertions(+), 0 deletions(-) > create mode 100644 programs/extrac32/Makefile.in > create mode 100644 programs/extrac32/extrac32.c > create mode 100644 programs/extrac32/parsecmd.c Hi, Andrey. The first spotted: --- +int PASCAL wWinMain ( HINSTANCE hInstance, HINSTANCE prev, LPWSTR cmdline, int show ) +{ ... + UINT j = 0; --- This initializer looks redundant --- +LPWSTR* ParseCmdline( LPCWSTR cmdline, int *argc ) + BOOL bQuote = FALSE; ... + i = j = k = z = nQuotes = 0; --- This is redundant for i and nQuotes at least.

1 0

Re: [3/4] user32/cursoricon.c: Multiple Fixes/Improvements for DrawIconEx
by Vitaliy Margolen Sept. 26, 2009

Sept. 26, 2009

Wilfried Pasquazzo wrote: > Best regards, > > Subject: Make DrawIconEx 2 additional tests (now passes all tests) > Subject: Make DrawIconEx pass a test You should really combine these 2 patches. You are modifying the same exact code and breaking it between patches (for some programs). Vitaliy.

1 0

"itune developement statues"
by Zhong Jiang Sept. 26, 2009

Sept. 26, 2009

Hi, I'm writing for the itunes progress and I've seen that the progress is rather on the negative side so I want to inquire on statues and is there anyway that I can help to further this progress.

1 0

Appinstall now supports standalone testing
by Austin English Sept. 26, 2009

Sept. 26, 2009

Howdy all, I've tweaked appinstall slightly so that anyone can run it without needing an ssh webserver to host the files. Simply do: $ wget http://winezeug.googlecode.com/svn/trunk/appinstall/appinstall.sh $ sh appinstall.sh That will download all the programs and run the tests. The results will be stored in `cwd`/appinstall-$date. Subsequent test runs of course will have the downloads cached, so they'll be much quicker. A couple things to note: you can change the tag name by doing: $ sh appinstall.sh tagname The result directory will then be `cwd`/appinstall-tagname. Lastly, you can override the version of wine used with: $ WINE=~/wine-git/wine sh appinstall.sh So now you have no excuse not to test your dangerous patches before sending to wine-patches ;-). -- -Austin

1 0

Re: [3/4] user32/cursoricon.c: Multiple Fixes/Improvements for DrawIconEx
by Juan Lang Sept. 26, 2009

Sept. 26, 2009

Hi Wilfried, minor comment on this patch: hBitTemp = SelectObject( hMemDC, hXorBits ); + + UINT bltFlag = 0; C89 doesn't allow you to mix code and declarations like this. You'll have to declare bltFlag at the top of the block. --Juan

1 0

Re: sti: test behavior of creation with the A/W functions versus STI_VERSION_FLAG_UNICODE
by Detlef Riekenberg Sept. 26, 2009

Sept. 26, 2009

On Fr, 2009-09-25 at 16:35 +0200, Damjan Jovanovic wrote: > * sti: test behavior of creation with the A/W functions versus > STI_VERSION_FLAG_UNICODE Using the name "sti_tests" is bad. The short "sti" is enough. You forgot the license notice We usually cast the result of GetProcAddress() as (void *). That is not perfect, but much better to read. -- By by ... Detlef

1 0

Re: [PATCH 2/4] mshtml: Add test suite for IHTMLLocation
by Detlef Riekenberg Sept. 26, 2009

Sept. 26, 2009

On Fr, 2009-09-25 at 15:30 -0500, Andrew Eikum wrote: > + todo_wine ok(SUCCEEDED(hres), "get_hash failed: 0x%08x\n", > hres); > + if(SUCCEEDED(hres)){ > + if(str_pass) > + ok(strcmp_wa_null(str, exp) == 0, "expected retrieved > hash to be L\"%s\", was: %s\n", exp, wine_dbgstr_w(str)); > + else > + todo_wine ok(strcmp_wa_null(str, exp) == 0, "expected > retrieved hash to be L\"%s\", was: %s\n", exp, wine_dbgstr_w(str)); > + } > +} > + Your Patch is hard to read. Please split the long lines (3 parts are easy) The whole patch can be split in smaller patches. > +static void perform_test(const struct location_test* test) In the case of a test failure or a crash, there is no chance to know, which test-set failed. A suffix (protocol name or line number) at the start of every ok() can be used to make the normal output as silent as possible. Using a trace before you call perform_tests() works also. That produce a bit noise in the result file, but the test binary is smaller. -- By by ... Detlef

1 0

Re: kernel32: check parameter of CheckRemoteDebuggerPresent with tests
by Alexandre Julliard Sept. 26, 2009

Sept. 26, 2009

Stefan Leichter <Stefan.Leichter(a)camline.com> writes: > + SetLastError(0xdeadbeef); > + bret = pCheckRemoteDebuggerPresent(INVALID_HANDLE_VALUE,NULL); > + ok(!bret , "expected CheckRemoteDebuggerPresent to fail\n"); > + ok(ERROR_INVALID_PARAMETER == GetLastError(), > + "expected error ERROR_INVALID_PARAMETER, got %d/%x\n",GetLastError(), GetLastError()); It doesn't make sense to pass INVALID_HANDLE_VALUE as a process handle, it's only for files. You probably want GetCurrentProcess(). -- Alexandre Julliard julliard(a)winehq.org

1 0

cppcheck sept 18 redux
by chris ahrendt Sept. 26, 2009

Sept. 26, 2009

False positive ticket created for: [/home/cahrendt/wine-git/dlls/wineps.drv/init.c:270]: (error) Possible null pointer dereference: dmW - otherwise it is redundant to check if dmW is null at line 272 what about the others? or the suggestion of adding an additional int to the array? chris

11 25

re: [3/6] ntdll: Move logic which computes file attributes to new function DIR_get_attributes() (take 4)
by Dan Kegel Sept. 25, 2009

Sept. 25, 2009

Hi Alexandre, thanks for committing the tests and cmd changes. What's left before the DIR_get_attribute change is up to snuff?

2 1

Ignore EINTR poll result code in WS_select
by Dmitriy Anisimkov Sept. 25, 2009

Sept. 25, 2009

Dear Wine Developers, Looks like such ignorance is usual practice in wine. I just need to ignore EINTR in one more place. Otherwise our client application is bad working in Wine.

4 4

Re: jscript: Update German translation
by Stefan Leichter Sept. 25, 2009

Sept. 25, 2009

Am Friday 25 September 2009 17:37:24 schrieb André Hentschel: > --- > dlls/jscript/jscript_De.rc | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) + IDS_JSCRIPT_EXPECTED "JScript bjekt erwartet" looks like you miss an "O" -- Stefan

1 0

Re: [4/7] comctl32/updown: Show that UDS_SETBUDDYINT switching doesn't affect behaviour for already created control
by Paul Vriens Sept. 25, 2009

Sept. 25, 2009

On 09/20/2009 10:02 PM, Nikolay Sivov wrote: >> From 8981901ce08501229487a1fcd2d430490cb66823 Mon Sep 17 00:00:00 2001 > From: Nikolay Sivov<bunglehead(a)gmail.com> > Date: Sun, 20 Sep 2009 02:52:39 +0400 > Subject: Tests to show that UDS_SETBUDDYINT switching doesn't affect behaviour for already created control > > --- > dlls/comctl32/tests/updown.c | 59 ++++++++++++++++++++++++++++++++++++----- > 1 files changed, 51 insertions(+), 8 deletions(-) > Hi Nikolay, This one (and also "[5/7] comctl32/updown: Fix control behaviour for UDS_SETBUDDYINT style") add some tests that use UDM_GETPOS32 and/or UDM_SETPOS32. These are only available as of 5.80. We are already skipping tests in updown because of this, but not these new ones yet. Any idea about the cleanest approach? -- Cheers, Paul.

2 3

Wine-Tango Updates
by Joel Holdsworth Sept. 25, 2009

Sept. 25, 2009

Hi All, Thanks to those of you who made comments about my Tango icon set. I've thought carefully about all the comments I've received, and I'm most of the way through the issues raised. I've revised idb_std_large.bmp, idb_std_small.bmp, floppy.ico, mycomputer.ico, netdrive.ico, netdrive2.ico, shortcut.ico, dxdiag, notepad, and wordpad. I still have one or two icons that I haven't yet touched: mydocs.ico, certwatermark.bmp, oic_hand.ico, iexplore, but I thought I'd share what I have while it's fresh in people's minds. Take a look http://www.airwebreathe.org.uk/wine-icon/ Let me know if you have more comments. Best Regards Joel

4 3

[bugzilla] donate to sponsoring bug
by sacchi antonio Sept. 25, 2009

Sept. 25, 2009

hello all. I'm new here but I'd like to suggest something, what do you think about this? http://img29.imageshack.us/img29/2940/wined.png Maybe it will be good for old/hard bugs? Thanks

6 9

Re: dlls/user32/edit.c -- remove superflous check
by Austin English Sept. 24, 2009

Sept. 24, 2009

On Thu, Sep 24, 2009 at 3:15 PM, todd baker <kkiddrrock707(a)yahoo.com> wrote: You forgot the patch. -- -Austin

1 0

Re: richedit: Silence fixme for WM_STYLECHANGING & WM_STYLECHANGED.
by James Mckenzie Sept. 24, 2009

Sept. 24, 2009

>From: Dylan Smith <dylan.ah.smith(a)gmail.com> >Sent: Sep 24, 2009 6:02 AM >To: wine-patches <wine-patches(a)winehq.org> >Subject: richedit: Silence fixme for WM_STYLECHANGING & WM_STYLECHANGED. > >I have tried changing the different styles, and have noticed that they >don't have any effect. The initial styles are the ones actually used, >which is why they are stored internally for the editor. Is this in Wine or in Windows? If it is in Wine, then we still need to implement the code to make the styles in the editor change. This may involve alot of work to get them completely functional as styles change at least one item. James McKenzie

2 1

Re: shell32: stub ShellHookProc for the Aion launcher
by Chris Robinson Sept. 24, 2009

Sept. 24, 2009

On Thursday 24 September 2009 11:54:33 am Aric Stewart wrote: > +@ stdcall ShellHookProc(long long long) ... > +LRESULT CALLBACK ShellHookProc(DWORD a, DWORD b, DWORD c) Doesn't CALLBACK signify cdecl, not stdcall?

2 1

Re: comctl32: initial theming tests
by Nikolay Sivov Sept. 24, 2009

Sept. 24, 2009

André Hentschel wrote: > tests class redirection and "load"-order atm > --- > dlls/comctl32/tests/Makefile.in | 1 + > dlls/comctl32/tests/theming.c | 301 +++++++++++++++++++++++++++++++++++++++ > 2 files changed, 302 insertions(+), 0 deletions(-) > create mode 100644 dlls/comctl32/tests/theming.c > Why not reuse this +static const CHAR manifest[] = from v6util.h. Also it's better to introduce new create/delete manifest helpers in this header and use them. It's better to use already defined classnames here: --- + struct classlist {WCHAR class[24]; BOOL todo;} listW[] = {like --- STATUSCLASSNAMEW for example. If the purpose of that --- static void test_redirection(HINSTANCE hInstance, LPCWSTR clsname, BOOL todo) --- is to show that message procedure moves to comctl32 after enabling themes than testing for comctl32 classes is useless - winproc is already in this module. I don't think it's a good choice: --- + pIsThemeActive = (void*)GetProcAddress(hUxtheme, "IsThemeActive"); + if (!pIsThemeActive || !pIsThemeActive()) --- I mean what if I don't use theming on XP but classic decorations will it return TRUE? Not sure but maybe this class reregistration occurs anyway when v6 module is loaded (manifest specified). Skipping like that will drop tests on native boxes that have theming disable what isn't good. BTW. Do you plan to reregister all user32 classes on comctl32 load? How could this be done at user32 side - testing for something like IsThemeActive() while loading user32?

3 2

Re: Fix for Bug 19851 (try 4) - Interlocked instruction support for ARM
by André Hentschel Sept. 24, 2009

Sept. 24, 2009

On Thu, Sep 24, 2009 at 12:19 PM, Colin Fowler <elethiomel(a)gmail.com> wrote: > Removed trailing white-spaces. I think this is good to go now :) every "else" still has trailing white-spaces and the first comment line("\* "). BTW i like that someone works on the ARM port. thats great! -- Best Regards, André Hentschel

1 0