Re: msxml3: Fix a failing test in win98

2008-09-03 Thread Paul Vriens
James Hawkins wrote: Hi, Changelog: * Fix a failing test in win98. dlls/msxml3/tests/xmlelem.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) Two things: The comment says Win98 returns

Re: msxml3: Fix a failing test in win98

2008-09-03 Thread James Hawkins
On Wed, Sep 3, 2008 at 1:05 AM, Paul Vriens [EMAIL PROTECTED] wrote: James Hawkins wrote: Hi, Changelog: * Fix a failing test in win98. dlls/msxml3/tests/xmlelem.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)

Re: [msxml3/tests 1/2] Use CreateFileA to please win98

2008-09-03 Thread James Hawkins
On Wed, Sep 3, 2008 at 2:54 AM, Paul Vriens [EMAIL PROTECTED] wrote: Hi, CreateFileW is not implemented on win98. This fixes the 2 remaining test failures for win98. (This is a replacement for the patch James sent). Changelog Use CreateFileA to please win98 Just a nitpick: +static

Re: [msxml3/tests 1/2] Use CreateFileA to please win98

2008-09-03 Thread Paul Vriens
James Hawkins wrote: On Wed, Sep 3, 2008 at 2:54 AM, Paul Vriens [EMAIL PROTECTED] wrote: Hi, CreateFileW is not implemented on win98. This fixes the 2 remaining test failures for win98. (This is a replacement for the patch James sent). Changelog Use CreateFileA to please win98 Just

Re: [9/9] gdiplus: initial GdipFlattenPath implementation

2008-09-03 Thread Alexandre Julliard
Nikolay Sivov [EMAIL PROTECTED] writes: +/* free all nodes including argument */ +void free_path_list(path_list_node_t *node) +{ +path_list_node_t *n = node; + +while(!n){ +GdipFree(node); +node = n = n-next; +} This won't do what you want. -- Alexandre

Re: ntdll: Fix a few failing tests in Vista

2008-09-03 Thread Alexandre Julliard
James Hawkins [EMAIL PROTECTED] writes: @@ -325,13 +327,21 @@ static void read_file_test(void) iosb.Information = 0xdeadbeef; offset.QuadPart = strlen(text) + 2; status = pNtReadFile( handle, event, apc, apc_count, iosb, buffer, 2, offset, NULL ); -ok( status ==

Re: [9/9] gdiplus: initial GdipFlattenPath implementation

2008-09-03 Thread Nikolay Sivov
Alexandre Julliard wrote: Nikolay Sivov [EMAIL PROTECTED] writes: +/* free all nodes including argument */ +void free_path_list(path_list_node_t *node) +{ +path_list_node_t *n = node; + +while(!n){ +GdipFree(node); +node = n = n-next; +} This

Re: Repeated messages (was Re: Adding Flawfinder to Patchwatcher)

2008-09-03 Thread Wesley Parish
On Tuesday 02 September 2008 07:33, Austin English wrote: On Sun, Aug 31, 2008 at 6:38 PM, Erik de Castro Lopo [EMAIL PROTECTED] wrote: Austin English wrote: I had a discussion with Dan about adding Flawfinder to the patchwatcher. Is anyone else getting multiple copies of this

Re: oleaut32: Add support for loading typelibs from NE files.

2008-09-03 Thread Rob Shearman
2008/9/2 Huw Davies [EMAIL PROTECTED]: --- a/dlls/oleaut32/typelib.c +++ b/dlls/oleaut32/typelib.c @@ -64,6 +64,7 @@ #include winnls.h #include winreg.h #include winuser.h +#include wine/winbase16.h #include wine/unicode.h #include objbase.h I don't think we should be including a

Re: wined3d: fix a memory leak and a performance regression in gen_arbfp_ffp_shader

2008-09-03 Thread Henri Verbeet
2008/9/3 Jeff Zaroyko [EMAIL PROTECTED]: Please ignore the previous patch sent two days ago, this one fixes a performance regression as well as removing the memory leak. -Jeff Do you have some kind of benchmark data for this change? I don't think allocating relatively large buffers like that

Re: wined3d: fix a memory leak and a performance regression in gen_arbfp_ffp_shader

2008-09-03 Thread Jeff Zaroyko
On Wed, Sep 3, 2008 at 10:25 PM, Henri Verbeet [EMAIL PROTECTED] wrote: 2008/9/3 Jeff Zaroyko [EMAIL PROTECTED]: Please ignore the previous patch sent two days ago, this one fixes a performance regression as well as removing the memory leak. -Jeff Do you have some kind of benchmark data for

Re: [PATCH 4/5] wined3d: Remove glDrawBuffer calls in BltOverride.

2008-09-03 Thread Alexander Dorofeyev
Henri Verbeet wrote: 2008/9/2 Alexander Dorofeyev [EMAIL PROTECTED]: I don't think this is possible yet, the variable is also used for some window/screen and upside-down coordinate magic. I do suspect that the code doing so may need to be fixed or removed, because it looks inconsistent with

Re: wined3d: fix a memory leak and a performance regression in gen_arbfp_ffp_shader

2008-09-03 Thread Alexandre Julliard
Jeff Zaroyko [EMAIL PROTECTED] writes: None as such since difference in performance is so large. I can say that without a doubt using HeapAlloc and HeapFree in this function causes noticeable delays such that in Battlefield 1942 the framerate is falls so much that it is unplayable, changing

Re: winhttp: Implement some more options.

2008-09-03 Thread Jacek Caban
Hi Hans, Hans Leidekker wrote: +static BOOL session_set_option( object_header_t *hdr, DWORD option, LPVOID buffer, DWORD buflen ) +{ +if (hdr-type != WINHTTP_HANDLE_TYPE_SESSION) +{ +release_object( hdr ); +set_last_error( ERROR_WINHTTP_INCORRECT_HANDLE_TYPE );

Re: winhttp: Implement some more options.

2008-09-03 Thread Hans Leidekker
On Wednesday 03 September 2008 16:37:48 Jacek Caban wrote: +static BOOL session_set_option( object_header_t *hdr, DWORD option, LPVOID buffer, DWORD buflen ) +{ +if (hdr-type != WINHTTP_HANDLE_TYPE_SESSION) +{ +release_object( hdr ); +set_last_error(

RE: wined3d: fix a memory leak and a performance regression in gen_arbfp_ffp_shader

2008-09-03 Thread Stefan Dösinger
The real problems are that the ffp description structure still contains 3 uninitialized padding bytes, and that a HeapFree is missing. I'm going to send patches for those

Re: [2/2] WineD3D: Make the srgb write flag bigger

2008-09-03 Thread Vitaliy Margolen
Stefan Dösinger wrote: Apparently gcc pads the structure size to DWORD alignment, which leaves 3 padding bytes which trigger differences in the memset. This fixes memory leaks in the fragment pipeline replacement

wineboot: Basic RunOnceEx support

2008-09-03 Thread Johan Dahlin
Just enough to make ie6 setup work. I'm not receiving list mail, please CC me when replying. Johan diff --git a/programs/wineboot/Makefile.in b/programs/wineboot/Makefile.in index cdbab2e..3be7e22 100644 --- a/programs/wineboot/Makefile.in +++ b/programs/wineboot/Makefile.in @@ -4,7 +4,7 @@

Re: Recent msi/package tests failures

2008-09-03 Thread Austin English
On Tue, Sep 2, 2008 at 12:45 AM, Paul Vriens [EMAIL PROTECTED] wrote: James Hawkins wrote: On Mon, Sep 1, 2008 at 8:08 PM, James Mckenzie [EMAIL PROTECTED] wrote: One file per run is all that should be created, IMHO. No, zero log files should be created. This discussion is about reducing

Re: AcceptEx Patch set Sep.4

2008-09-03 Thread Scott Lindeneau
I have tried to make this new patch set as clean as possible. The previous set made numerous changes to many parts of the wineserver. This set attempts to only make modifications to the sock.c functions. Most of it is simply added functionality which is, in my opinion, required to have a correct

intermittent ddraw/visual crash

2008-09-03 Thread Dan Kegel
I'm seeing a crash occasionally in ddraw/tests/visual.c on hardy, with a cheap PNY 8500 GT 512MB PCIe nvidia card. (Hrm. http://www.overclockersclub.com/reviews/8500gt/11.htm has a review that says stay away from this one, maybe I shouldn't have gone for the cheapest card.) Out of 200 runs, I

Re: wineboot: Basic RunOnceEx support

2008-09-03 Thread Dmitry Timoshkov
Johan Dahlin [EMAIL PROTECTED] wrote: +typedef HRESULT (*DLLREGISTER) (void); +typedef HRESULT (*DLLINSTALL) (BOOL,LPCWSTR); Wrong calling convention. +static int RunDllRegisterServer(WCHAR* strDll) +{ +DLLREGISTER pfRegister = NULL; +HRESULT hr; +HMODULE

RE: [2/2] WineD3D: Make the srgb write flag bigger

2008-09-03 Thread Stefan Dösinger
You should add a note to the source about this. Or someone might change the size of this structure and the old bug will return. That's probably a good idea However the best way to fix this problem would be to pack this structure: #include pshpack1.h struct ffp_settings { ... };

Re: [2/2] WineD3D: Make the srgb write flag bigger

2008-09-03 Thread Vitaliy Margolen
Stefan Dösinger wrote: You should add a note to the source about this. Or someone might change the size of this structure and the old bug will return. That's probably a good idea However the best way to fix this problem would be to pack this structure: #include pshpack1.h struct

Re: intermittent ddraw/visual crash

2008-09-03 Thread Henri Verbeet
2008/9/4 Dan Kegel [EMAIL PROTECTED]: I'm seeing a crash occasionally in ddraw/tests/visual.c on hardy, with a cheap PNY 8500 GT 512MB PCIe nvidia card. (Hrm. http://www.overclockersclub.com/reviews/8500gt/11.htm has a review that says stay away from this one, maybe I shouldn't have gone for

Re: Recent msi/package tests failures

2008-09-03 Thread Paul Vriens
Austin English wrote: On Tue, Sep 2, 2008 at 12:45 AM, Paul Vriens [EMAIL PROTECTED] wrote: James Hawkins wrote: On Mon, Sep 1, 2008 at 8:08 PM, James Mckenzie [EMAIL PROTECTED] wrote: One file per run is all that should be created, IMHO. No, zero log files should be created. This