- wine-devel - winehq.org

Re: cmd: Disable Ctrl-C interrupt
by Jason Edmeades 06 Jun '13

06 Jun '13

Hi Hugh, Thanks for looking at this issue with wine's cmd I do not think this patch is enough - From memory (I once tried something similar) the problem with resolving it the way you have is you then lose the ability of breaking out of a batch program. I think the right solution needs an actual handler which resets back to the command prompt mode, optionally asking something like whether you want to end the program or not (with a no continuing from where it left off, with consideration around for loops etc!). I would suggest it's better to allow control c for now rather than get into situations you cannot terminate Jason

2 1

Re: oledb32: Correct DBPROP_INIT_PROVIDERSTRING and DBPROP_INIT_HWND types
by Nikolay Sivov 06 Jun '13

06 Jun '13

On Thu, Jun 6, 2013 at 9:55 AM, Alistair Leslie-Hughes < leslie_alistair(a)hotmail.com> wrote: > Hi, > Corrects a spelling mistake too and > > Changelog: > oledb32: Correct DBPROP_INIT_PROVIDERSTRING and DBPROP_INIT_HWND types > > > Best Regards > Alistair Leslie-Hughes > > > > - { extemdedW, DBPROP_INIT_PROVIDERSTRING, DBPROPOPTIONS_REQUIRED, VT_BSTR }, + { extemdedW, DBPROP_INIT_PROVIDERSTRING, DBPROPOPTIONS_OPTIONAL, VT_BSTR }, { gen_timeout, DBPROP_INIT_GENERALTIMEOUT, DBPROPOPTIONS_OPTIONAL, VT_I4 }, - { initcatW, DBPROP_CATALOGLOCATION, DBPROPOPTIONS_OPTIONAL, VT_BSTR }, + { initcatW, DBPROP_CATALOGLOCATION, DBPROPOPTIONS_OPTIONAL, VT_I4 }, How it's an integer? Catalog location is a path basically. Regarding provider string, could you please add a test for this flag?

1 0

Re: [PATCH 2/3] oleaut32/tests: Add save/load tests for created typelibs
by Marvin 05 Jun '13

05 Jun '13

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=25849 Your paranoid android. === WXPX64 (64 bit typelib) === typelib.c:2549: Test failed: cbSizeInstance = 8 typelib.c:2554: Test failed: cbSizeVft = 128 typelib.c:2556: Test failed: cbAlignment = 8 typelib.c:2631: Test failed: got 32 typelib.c:2657: Test failed: got 40 typelib.c:2682: Test failed: got 48 typelib.c:2743: Test failed: got 56 typelib.c:2791: Test failed: got 64 typelib.c:2830: Test failed: got 72 typelib.c:2869: Test failed: got 80 typelib.c:2911: Test failed: got 88 typelib.c:2947: Test failed: got 96 typelib.c:2977: Test failed: got 104 typelib.c:3011: Test failed: got 112 typelib.c:3044: Test failed: got 120 typelib.c:3080: Test failed: cbSizeInstance = 8 typelib.c:3085: Test failed: cbSizeVft = 21856 typelib.c:3086: Test failed: cbAlignment = 8 typelib.c:3105: Test failed: got 5550 typelib.c:3137: Test failed: got 5558 typelib.c:3169: Test failed: cbSizeInstance = 8 typelib.c:3175: Test failed: cbAlignment = 8 typelib.c:3196: Test failed: cbSizeInstance = 8 typelib.c:3202: Test failed: cbAlignment = 8 typelib.c:3242: Test failed: cbSizeInstance = 8 typelib.c:3247: Test failed: cbSizeVft = 56 typelib.c:3248: Test failed: cbAlignment = 8 typelib.c:3263: Test failed: cbSizeInstance = 8 typelib.c:3268: Test failed: cbSizeVft = 64 typelib.c:3269: Test failed: cbAlignment = 8 === WVISTAX64 (64 bit typelib) === typelib.c:2549: Test failed: cbSizeInstance = 8 typelib.c:2554: Test failed: cbSizeVft = 128 typelib.c:2556: Test failed: cbAlignment = 8 typelib.c:2631: Test failed: got 32 typelib.c:2657: Test failed: got 40 typelib.c:2682: Test failed: got 48 typelib.c:2743: Test failed: got 56 typelib.c:2791: Test failed: got 64 typelib.c:2830: Test failed: got 72 typelib.c:2869: Test failed: got 80 typelib.c:2911: Test failed: got 88 typelib.c:2947: Test failed: got 96 typelib.c:2977: Test failed: got 104 typelib.c:3011: Test failed: got 112 typelib.c:3044: Test failed: got 120 typelib.c:3080: Test failed: cbSizeInstance = 8 typelib.c:3085: Test failed: cbSizeVft = 21856 typelib.c:3086: Test failed: cbAlignment = 8 typelib.c:3105: Test failed: got 5550 typelib.c:3137: Test failed: got 5558 typelib.c:3169: Test failed: cbSizeInstance = 8 typelib.c:3175: Test failed: cbAlignment = 8 typelib.c:3196: Test failed: cbSizeInstance = 8 typelib.c:3202: Test failed: cbAlignment = 8 typelib.c:3242: Test failed: cbSizeInstance = 8 typelib.c:3247: Test failed: cbSizeVft = 56 typelib.c:3248: Test failed: cbAlignment = 8 typelib.c:3263: Test failed: cbSizeInstance = 8 typelib.c:3268: Test failed: cbSizeVft = 64 typelib.c:3269: Test failed: cbAlignment = 8 === W7PROX64 (64 bit typelib) === typelib.c:2549: Test failed: cbSizeInstance = 8 typelib.c:2554: Test failed: cbSizeVft = 128 typelib.c:2556: Test failed: cbAlignment = 8 typelib.c:2631: Test failed: got 32 typelib.c:2657: Test failed: got 40 typelib.c:2682: Test failed: got 48 typelib.c:2743: Test failed: got 56 typelib.c:2791: Test failed: got 64 typelib.c:2830: Test failed: got 72 typelib.c:2869: Test failed: got 80 typelib.c:2911: Test failed: got 88 typelib.c:2947: Test failed: got 96 typelib.c:2977: Test failed: got 104 typelib.c:3011: Test failed: got 112 typelib.c:3044: Test failed: got 120 typelib.c:3080: Test failed: cbSizeInstance = 8 typelib.c:3085: Test failed: cbSizeVft = 21856 typelib.c:3086: Test failed: cbAlignment = 8 typelib.c:3105: Test failed: got 5550 typelib.c:3137: Test failed: got 5558 typelib.c:3169: Test failed: cbSizeInstance = 8 typelib.c:3175: Test failed: cbAlignment = 8 typelib.c:3196: Test failed: cbSizeInstance = 8 typelib.c:3202: Test failed: cbAlignment = 8 typelib.c:3242: Test failed: cbSizeInstance = 8 typelib.c:3247: Test failed: cbSizeVft = 56 typelib.c:3248: Test failed: cbAlignment = 8 typelib.c:3263: Test failed: cbSizeInstance = 8 typelib.c:3268: Test failed: cbSizeVft = 64 typelib.c:3269: Test failed: cbAlignment = 8 === TEST64_W7SP1 (64 bit typelib) === typelib.c:2549: Test failed: cbSizeInstance = 8 typelib.c:2554: Test failed: cbSizeVft = 128 typelib.c:2556: Test failed: cbAlignment = 8 typelib.c:2631: Test failed: got 32 typelib.c:2657: Test failed: got 40 typelib.c:2682: Test failed: got 48 typelib.c:2743: Test failed: got 56 typelib.c:2791: Test failed: got 64 typelib.c:2830: Test failed: got 72 typelib.c:2869: Test failed: got 80 typelib.c:2911: Test failed: got 88 typelib.c:2947: Test failed: got 96 typelib.c:2977: Test failed: got 104 typelib.c:3011: Test failed: got 112 typelib.c:3044: Test failed: got 120 typelib.c:3080: Test failed: cbSizeInstance = 8 typelib.c:3085: Test failed: cbSizeVft = 21856 typelib.c:3086: Test failed: cbAlignment = 8 typelib.c:3105: Test failed: got 5550 typelib.c:3137: Test failed: got 5558 typelib.c:3169: Test failed: cbSizeInstance = 8 typelib.c:3175: Test failed: cbAlignment = 8 typelib.c:3196: Test failed: cbSizeInstance = 8 typelib.c:3202: Test failed: cbAlignment = 8 typelib.c:3242: Test failed: cbSizeInstance = 8 typelib.c:3247: Test failed: cbSizeVft = 56 typelib.c:3248: Test failed: cbAlignment = 8 typelib.c:3263: Test failed: cbSizeInstance = 8 typelib.c:3268: Test failed: cbSizeVft = 64 typelib.c:3269: Test failed: cbAlignment = 8

1 0

Re: ole32: Add a basic OLE client/server test suite. Take 2.
by Dmitry Timoshkov 05 Jun '13

05 Jun '13

Dmitry Timoshkov <dmitry(a)baikal.ru> wrote: > With minor clean ups and with increased timeout to wait for server termination > to please some really slow VMs (that allowed to remove broken() statements). It would be helpful to provide some feedback and explain the 'pending' state of the patch which adds a test. -- Dmitry.

2 7

Re: [PATCH 3/5] wined3d: Pass a wined3d_resource_desc structure to wined3d_texture_create_cube().
by Stefan Dösinger 05 Jun '13

05 Jun '13

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am 2013-06-05 11:06, schrieb Henri Verbeet: > + desc.width = edge_length; + desc.height = edge_length; + > desc.depth = 1; Wouldn't it make sense to either set desc.height = 1 or desc.depth = edge_length? -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRrwNtAAoJEN0/YqbEcdMwLkoP/jN3ooH3ur4ZLMgSc96VT+WR b+EcwCmK1yigPrY04UPiHmSVSS/g9bgR/BsVoQB9d6Fa2UfFeDEGGgOYdVOE6/9P guRWs6RAMTv4AfuDCjmRC3h2fO0nigg3sLkmvqG5qZF8hmGbItijB5zREWHys3Bk kedPMQaWXWuq9wwHa7ZM1B3wcMiZsxqfcyitFa1FsB7LvbLHIYuMzNQrViObGvT5 75J1Ux/WmFu9ZKTvWjIyFzhWg9oqeHPpB1lXgj4w3ONkOa0Rw27EUsxHRTlGiYm+ 4qGyvclsE1Ep2A2NK/kCUU1SuxeSP/bW38P9Lc8J/xNb8Si8zNu2VApTYjOIOk9q 3wHKgU0MuRRnw5ophMM9xNhIT/lt+0FilNZYosvEul+A0tAhncthCege0ZBRIeSf ZKlRGrInBMSy59BJwxVvPxaF702M0xHdkpKEMnML1NYCRg9+gbm862y/KRczvBxL kRWs6AIBr0247voTHvhRBJTtEanCeKP+qGLoatNVimPFaEDnSj9xxHIA+4XaThPW 2A+WjSPc7ni0Z8IKGQB9hOku0ZKvX2BJIAXiFm1KQW23psqUHF5CTR3w48yG7Dmh pm8SETIud6SwakKTW5ikp5ObjLYdM1Z8M8hNIkU64O8k7+aSX3hAqKW2shC6d/xm REcwzCU6NhmbDWgC5epH =tH28 -----END PGP SIGNATURE-----

2 1

Re: [PATCH] d3dx9_36: Implement ID3DXSkinInfoImpl_UpdateSkinnedMesh.
by Rico Schüller 05 Jun '13

05 Jun '13

On 05.06.2013 08:48, Christian Costa wrote: > bone = &This->bones[bone_num]; > + /* Decrement refcounts of vertices previously influenced */ > + for (i = 0; i < bone->num_influences; i++) > + if (bone->vertices[i] < This->num_vertices) /* Ignore out of range value */ > + This->vertices_refcounts[bone->vertices[i]]--; > bone->num_influences = num_influences; > HeapFree(GetProcessHeap(), 0, bone->vertices); > HeapFree(GetProcessHeap(), 0, bone->weights); > bone->vertices = new_vertices; > bone->weights = new_weights; > + /* Increment refcounts of vertices newly influenced */ > + for (i = 0; i < bone->num_influences; i++) > + { > + if (bone->vertices[i] < This->num_vertices) /* Ignore out of range value */ > + This->vertices_refcounts[bone->vertices[i]]--; > + } Please have a look at the code style (brackets {} at the for loops?). It should be a bit more consistent. Also I think an increment (as said in the comment) is something like "This->vertices_refcounts[bone->vertices[i]]++;". Maybe you could add a test to verify if the comment or the code is correct? Cheers Rico

2 1

Re: [PATCH 1/2] server: Don't create new handle when DUP_HANDLE_CLOSE_SOURCE is used if possible (try2)
by Alexandre Julliard 05 Jun '13

05 Jun '13

Piotr Caban <piotr(a)codeweavers.com> writes: > @@ -890,6 +890,7 @@ struct rawinput_device > obj_handle_t handle; /* duplicated handle in dst process */ > int self; /* is the source the current process? */ > int closed; /* whether the source handle has been closed */ > + int reused; /* whether the source handle has been reused */ I don't think you need an extra flag, there's nothing the client has to do differently in that case (the warning is mostly cosmetic anyway). -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: api-ms-win-core-localregistry-l1-1-0: add stub dll
by Charles Davis 05 Jun '13

05 Jun '13

On Jun 4, 2013, at 12:15 PM, Austin English wrote: > diff --git a/dlls/api-ms-win-core-localregistry-l1-1-0/api-ms-win-core-localregistry-l1-1-0.spec b/dlls/api-ms-win-core-localregistry-l1-1-0/api-ms-win-core-localregistry-l1-1-0.spec > new file mode 100644 > index 0000000..cc15d4e > --- /dev/null > +++ b/dlls/api-ms-win-core-localregistry-l1-1-0/api-ms-win-core-localregistry-l1-1-0.spec > @@ -0,0 +1,40 @@ > +@ stub RegCloseKey Why don't you just forward them to their implementations in advapi32? Is there something oddly different about these functions that I don't know about? Chip

2 1

Re: bugzilla: Change no bugs found message
by Christian Costa 04 Jun '13

04 Jun '13

Hi, Minor but shouldn't it be "bug" instead "bugs"? Christian

2 2

Re: [PATCH 1/8] winemac: Don't unminimize a window on first activation during start-up.
by Dmitry Timoshkov 04 Jun '13

04 Jun '13

Ken Thomases <ken(a)codeweavers.com> wrote: > --- > dlls/winemac.drv/cocoa_app.h | 2 ++ > dlls/winemac.drv/cocoa_app.m | 3 ++- > 2 files changed, 4 insertions(+), 1 deletions(-) I wonder, do you actually run 'make test' in dlls/user32/tests with your changes? My guess is that you don't. -- Dmitry.

2 1

Re: [PATCH] winhttp: initialize schemes (Coverity) (resubmit)
by Hans Leidekker 04 Jun '13

04 Jun '13

On Tue, 2013-06-04 at 10:16 +0200, Marcus Meissner wrote: > 1020943 Uninitialized scalar variable > > After suggestion by Hans, do it in the helper ... especially > as the other external callers do not initialize schemes either. I also said that you shouldn't touch the parameter on error :) The other callers pass a pointer to a user variable and the attached test confirms that native doesn't touch it.

1 0

testbot and NT4
by Ann and Jason Edmeades 04 Jun '13

04 Jun '13

Hiya, Its been a while since I did some cmd work and noticed there were some new bugs so am doing some fiddling. The real pain for me was always keeping the tests passing on nt4, so I noted with relief/surprise that the testbot doesnt have an NT4 VM anymore. Is it correct that there is no longer a requirement for tests to pass on NT4 for submission (if it's required, I really need a nt4 testbot!) If not, is there any objection to removing the numerous hacks in the test suite so they run on NT4? Thanks, Jason

6 6

Re: [PATCH] mscms: Use lcms2, when available [try 2]
by Hans Leidekker 04 Jun '13

04 Jun '13

On Sun, 2013-06-02 at 22:49 +0200, Detlef Riekenberg wrote: > try2: > import the unchanged icc34.h from the lcms (v1) dev. package. > (Do we need to add a Wine LGPL licence header for icc34.h?) > > The plan is to remove this include, when changing Wine > to use the new getter/setter functions of lcms2 > (After the release of Wine-1.6) My plan was to switch to lcms2 soon *after* the release of Wine 1.6. I studied the API some more and my conclusion is that while it doesn't have all we need (specifically cmsReadRawTag and cmsWriteRawTag don't do what we want), we can avoid importing the icc34.h header. I'm not in favor of adding temporary workarounds. If distributions want to drop support for lcms1 in 1.6 I'd suggest that they backport the patches.

3 4

Re: Add tests for ntdll event functions
by Marvin 04 Jun '13

04 Jun '13

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=25813 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: Add tests for ntdll event functions
by Andrew Cook 04 Jun '13

04 Jun '13

While implemeting NtQueryEvent i stumbled across an odd permissions issue, which is really what this patch tests if ((event = create_event( root, &name, req->attributes, req->manual_reset, req->initial_state, sd ))) { if (get_error() == STATUS_OBJECT_NAME_EXISTS) reply->handle = alloc_handle( current->process, event, req->access, req->attributes ); else reply->handle = alloc_handle_no_access_check( current->process, event, req->access, req->attributes ); release_object( event ); } when creating an event, the handle is allocated with alloc_handle_no_access_check, which does not call event_map_access this causes event_op and event_query to fail, as the handle only has GENERIC_ALL, not the actual event permissions, which does not match windows behaviour (or at least windows 7 anyway), On 04/06/13 11:13, Andrew Cook wrote: > --- > dlls/ntdll/tests/Makefile.in | 1 + > dlls/ntdll/tests/event.c | 69 > ++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 70 insertions(+) > create mode 100644 dlls/ntdll/tests/event.c > > diff --git a/dlls/ntdll/tests/Makefile.in b/dlls/ntdll/tests/Makefile.in > index 10d6674..a44b880 100644 > --- a/dlls/ntdll/tests/Makefile.in > +++ b/dlls/ntdll/tests/Makefile.in > @@ -7,6 +7,7 @@ C_SRCS = \ > directory.c \ > env.c \ > error.c \ > + event.c \ > exception.c \ > file.c \ > generated.c \ > diff --git a/dlls/ntdll/tests/event.c b/dlls/ntdll/tests/event.c > new file mode 100644 > index 0000000..f29abc3 > --- /dev/null > +++ b/dlls/ntdll/tests/event.c > @@ -0,0 +1,69 @@ > +/* > + * Unit test suite for event functions > + * > + * Copyright 2005 Robert Shearman > + * Copyright 2005 Vitaliy Margolen > + * > + * This library is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * This library is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this library; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > 02110-1301, USA > + */ > + > +#include "ntdll_test.h" > +#include "winternl.h" > +#include "winnt.h" > + > +static NTSTATUS (WINAPI *pNtCreateEvent) ( PHANDLE, ACCESS_MASK, const > POBJECT_ATTRIBUTES, BOOLEAN, BOOLEAN); > +static NTSTATUS (WINAPI *pNtPulseEvent) ( HANDLE, PULONG ); > +static NTSTATUS (WINAPI *pNtQueryEvent) ( HANDLE, > EVENT_INFORMATION_CLASS, PVOID, ULONG, PULONG ); > +static NTSTATUS (WINAPI *pNtClose) ( HANDLE ); > +static VOID (WINAPI *pRtlInitUnicodeString)( PUNICODE_STRING, > LPCWSTR ); > + > +START_TEST(event) > +{ > + HANDLE Event; > + NTSTATUS status; > + UNICODE_STRING str; > + OBJECT_ATTRIBUTES attr; > + EVENT_BASIC_INFORMATION info; > + HMODULE hntdll = GetModuleHandleA("ntdll.dll"); > + HMODULE hkernel32 = GetModuleHandleA("kernel32.dll"); > + static const WCHAR eventName[] = > {'\\','B','a','s','e','N','a','m','e','d','O','b','j','e','c','t','s','\\','t','e','s','t','E','v','e','n','t',0}; > + > + if (!hntdll) > + { > + skip("not running on NT, skipping test\n"); > + return; > + } > + > + pNtCreateEvent = (void *)GetProcAddress(hntdll, "NtCreateEvent"); > + pNtQueryEvent = (void *)GetProcAddress(hntdll, "NtQueryEvent"); > + pNtPulseEvent = (void *)GetProcAddress(hntdll, "NtPulseEvent"); > + pNtClose = (void *)GetProcAddress(hntdll, "NtClose"); > + pRtlInitUnicodeString = (void *)GetProcAddress(hntdll, > "RtlInitUnicodeString"); > + > + pRtlInitUnicodeString(&str, eventName); > + InitializeObjectAttributes(&attr, &str, 0, 0, NULL); > + > + status = pNtCreateEvent(&Event, GENERIC_ALL, &attr, 0, 0); > + ok( status == STATUS_SUCCESS, "NtCreateEvent failed %08x\n", status ); > + > + status = pNtPulseEvent(Event, 0); > + todo_wine ok( status == STATUS_SUCCESS, "NtPulseEvent failed > %08x\n", status ); > + > + status = pNtQueryEvent(Event, EventBasicInformation, &info, > sizeof(info), NULL); > + todo_wine ok( status == STATUS_SUCCESS, "NtQueryEvent failed > %08x\n", status ); > + > + status = pNtClose(Event); > + ok( status == STATUS_SUCCESS, "NtClose failed %08x\n", status ); > +}

1 0

Re: How to stop gecko and mono popups while git bisecting?
by Dan Kegel 03 Jun '13

03 Jun '13

Or run https://winezeug.googlecode.com/svn/trunk/install-addons.sh

1 0

How to stop gecko and mono popups while git bisecting?
by Alan W. Irwin 03 Jun '13

03 Jun '13

I just finished a git bisect to establish the cause of the git regression given by http://bugs.winehq.org/show_bug.cgi?id=33715. My detailed report concerning the result of the bisect appears there, but I have an additional question concerning the git bisect process itself. I pretty much follow the recommendations in http://wiki.winehq.org/RegressionTesting, but I like to completely remove my WINEPREFIX directory as the first step in my bisect test script to eliminate the possibility that some unclean cached result there is causing issues. But after a particular version of wine is built, that test script goes on to test it using wineconsole which has to populate WINEPREFIX itself (which is fine) since I removed it for a reason, but that step is accompanied by some popups concerning what I want to do about downloading mono (not available on Debian testing) and gecko (available on Debian testing, but I haven't figured out how to make the ./configure step of the wine build recognize that). I don't mind the popups, but I do mind having to answer the questions posed by them (say through the night) as the automated git bisect proceeds. For the case where wineconsole populates the WINEPREFIX directory for the first time, is there a way to configure wine or run wineconsole so those popups that require me answering a question can be eliminated? Alan __________________________ Alan W. Irwin Astronomical research affiliation with Department of Physics and Astronomy, University of Victoria (astrowww.phys.uvic.ca). Programming affiliations with the FreeEOS equation-of-state implementation for stellar interiors (freeeos.sf.net); the Time Ephemerides project (timeephem.sf.net); PLplot scientific plotting software package (plplot.sf.net); the libLASi project (unifont.org/lasi); the Loads of Linux Links project (loll.sf.net); and the Linux Brochure Project (lbproject.sf.net). __________________________ Linux-powered Science __________________________

2 2

Re: dinput: Remove error-prone new joystick code
by Alexandre Julliard 03 Jun '13

03 Jun '13

"Bassi, Gurmail" <Gurmail.Bassi(a)mottmac.com> writes: > The joystick code I submitted in Wine patch 95617 seems to be causing problems to some users with both hardware (Wacom Cintiq) and software (Lord of the Rings Online) - see Wine bugs 33496 & 33593 > > This patch removes the offending code, until the cause can be fully investigated and a new patch issued if necessary Please fix the code instead of removing it. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 2/2] kernet32: Add DuplicateHandle tests with DUPLICATE_CLOSE_SOURCE flag
by Alexandre Julliard 03 Jun '13

03 Jun '13

Piotr Caban <piotr(a)codeweavers.com> writes: > --- > dlls/kernel32/tests/process.c | 83 > +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 83 insertions(+) You should probably add some tests that first close lower-numbered handles, to see if it may not be simply allocating the first free one instead. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [1/2] DWrite: Implemented functions regarding text alignment. (try 3)
by Nikolay Sivov 31 May '13

31 May '13

On Fri, May 31, 2013 at 9:07 PM, Fabian Müntefering <Cookiezzz97(a)gmx.de>wrote: > Implemented functions of text format and text layout regarding text > alignment. > > Try 3: Used proper enum value to set the enum variables. > > > > > + if(alignment!=DWRITE_TEXT_ALIGNMENT_CENTER && alignment!=DWRITE_TEXT_ALIGNMENT_LEADING && + alignment!=DWRITE_TEXT_ALIGNMENT_JUSTIFIED && alignment!=DWRITE_TEXT_ALIGNMENT_TRAILING) + return E_INVALIDARG; + This->format.text_align = alignment; This doesn't look great, you'll need a test that shows it really rejects meaningless values, and if it does it would be simpler to compare with greatest enum value.

1 0

Re: [PATCH] slc: Fix return type
by Christian Costa 31 May '13

31 May '13

Le 29/05/2013 12:06, Detlef Riekenberg a écrit : > -- > By by ... Detlef > --- > dlls/slc/slc.c | 31 +++++++++++++++++++++++-------- > include/slpublic.h | 2 +- > 2 files changed, 24 insertions(+), 9 deletions(-) > > diff --git a/dlls/slc/slc.c b/dlls/slc/slc.c > index e7416f1..7fc9b93 100644 > --- a/dlls/slc/slc.c > +++ b/dlls/slc/slc.c > @@ -1,6 +1,7 @@ > /* > * > * Copyright 2008 Alistair Leslie-Hughes > + * Copyright 2010 Detlef Riekenberg > * > * This library is free software; you can redistribute it and/or > * modify it under the terms of the GNU Lesser General Public > @@ -28,15 +29,8 @@ > > WINE_DEFAULT_DEBUG_CHANNEL(slc); > > -DWORD WINAPI SLGetWindowsInformationDWORD(LPCWSTR lpszValueName, LPDWORD pdwValue) > -{ > - FIXME("(%s) stub\n", debugstr_w(lpszValueName) ); > - > - return SL_E_RIGHT_NOT_GRANTED; > -} > - > /*********************************************************************** > - * DllMain (CLUSAPI.@) > + * DllMain (slc.@) > * > */ > BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) > @@ -51,3 +45,24 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) > } > return TRUE; > } > + > +/*********************************************************************** > + * SLGetWindowsInformationDWORD (slc.@) > + * > + * check the license and return the value for the given name > + * > + * PARAMS > + * name [i] application related value name to check > + * pvalue [o] ptr, where the value for the given name should be saved > + * > + * RETURNS > + * Success: S_OK > + * Failure: A HRESULT error code > + * > + */ > +HRESULT WINAPI SLGetWindowsInformationDWORD(PCWSTR name, DWORD *pvalue) > +{ > + FIXME("(%s, %p) stub\n", debugstr_w(name), pvalue); > + > + return SL_E_RIGHT_NOT_GRANTED; > +} > diff --git a/include/slpublic.h b/include/slpublic.h > index a2c6d42..5553787 100644 > --- a/include/slpublic.h > +++ b/include/slpublic.h > @@ -29,7 +29,7 @@ extern "C" { > #define SLCAPI DECLSPEC_IMPORT > #endif > > -SLCAPI DWORD WINAPI SLGetWindowsInformationDWORD(LPCWSTR lpszValueName, LPDWORD pdwValue); > +SLCAPI HRESULT WINAPI SLGetWindowsInformationDWORD(PCWSTR, DWORD *); > > > #ifdef __cplusplus I think s/PCWSTR/const WCHAR */ and s/pvalue/value/ would be better. :)

2 2

Re: [PATCH] winmm/tests: Fix test that fails on NT4
by Marvin 31 May '13

31 May '13

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=25788 Your paranoid android. === WINEBUILD (build) === Make failed

1 0

Re: [1/2] explorer: Initial implementation of desktop launchers.
by Qian Hong 31 May '13

31 May '13

Hello Hans, On Tue, May 28, 2013 at 6:42 PM, Hans Leidekker <hans(a)codeweavers.com> wrote: > + hr = SHGetKnownFolderPath( &FOLDERID_Desktop, 0, NULL, &desktop_folder ); > + if (FAILED( hr )) > + { > + WINE_ERR("Could not get user desktop folder\n"); > + return; > + } > + hr = SHGetKnownFolderPath( &FOLDERID_PublicDesktop, 0, NULL, &desktop_folder_public ); > + if (FAILED( hr )) > + { > + WINE_ERR("Could not get public desktop folder\n"); > + CoTaskMemFree( desktop_folder ); > + return; > + } Does KF_FLAG_CREATE look better here? Other wise every time a new wineprefix created there is an err output. Maybe I'm missing something here, thanks for your work. -- Regards, Qian Hong - http://www.winehq.org

2 1

Re: [PATCH 2/4] d3dxof: Move list type and list nb elements into the parse context.
by Alexandre Julliard 30 May '13

30 May '13

Christian Costa <titan.costa(a)gmail.com> writes: > --- > dlls/d3dxof/d3dxof_private.h | 2 ++ > dlls/d3dxof/parsing.c | 23 ++++++++++------------- > 2 files changed, 12 insertions(+), 13 deletions(-) It doesn't work: ../../../tools/runtest -q -P wine -M d3dx9_36.dll -T ../../.. -p d3dx9_36_test.exe.so mesh.c && touch mesh.ok mesh.c:2291: Test failed: Expected E_FAIL, got 0x88760390 mesh.c:2299: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2323: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2347: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2399: Test failed: Expected E_FAIL, got 0x88760390 mesh.c:2403: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2407: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2408: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2409: Test failed: Expected D3D_OK, got 0x88760390 mesh.c:2495: Tests skipped: D3DXCreateBox failed make: *** [mesh.ok] Error 9 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/2] advpack: Handle quoted arguments to rundll exports.
by Alexandre Julliard 30 May '13

30 May '13

Vincent Povirk <madewokherd(a)gmail.com> writes: > @@ -205,13 +205,24 @@ static HRESULT run_setup_commands_callback(HINF hinf, PCWSTR field, const void * > > /* sequentially returns pointers to parameters in a parameter list > * returns NULL if the parameter is empty, e.g. one,,three */ > -LPWSTR get_parameter(LPWSTR *params, WCHAR separator) > +LPWSTR get_parameter(LPWSTR *params, WCHAR separator, int quoted) Using proper boolean type and constants would be preferable. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/3] DWrite: Implemented functions regarding text alignment. (try 2)
by Alexandre Julliard 30 May '13

30 May '13

Fabian Müntefering <Cookiezzz97(a)gmx.de> writes: > @@ -1023,6 +1034,9 @@ HRESULT create_textformat(const WCHAR *family_name, IDWriteFontCollection *colle > This->format.size = size; > This->format.stretch = stretch; > > + /*Default values.*/ > + This->format.text_align = 0; Please use proper enum values when setting enum variables. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [4/4] kernel32: Don't move fixed memory area in GlobalReAlloc. (try 3)
by Marvin 30 May '13

30 May '13

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=25774 Your paranoid android. === WVISTAX64 (64 bit heap) === No test summary line found === W7PROX64 (64 bit heap) === No test summary line found === TEST64_W7SP1 (64 bit heap) === No test summary line found

1 0

Re: [1/4] kernel32: Add more GlobalReAlloc/LocalReAlloc tests.
by Marvin 30 May '13

30 May '13

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=25773 Your paranoid android. === WVISTAX64 (64 bit heap) === No test summary line found === W7PROX64 (64 bit heap) === No test summary line found === TEST64_W7SP1 (64 bit heap) === No test summary line found

1 0

Re: msvcrt: remove hack
by Henri Verbeet 30 May '13

30 May '13

On 30 May 2013 07:23, Austin English <austinenglish(a)gmail.com> wrote: > and no modern gcc/other compiler that I can find cares. > It does if you add -Wunused-parameter, but then you'll get a lot of other warnings as well. IMO the "action = action;" line is pretty pointless and best just removed, though if people are just looking for a different workaround, "(void)action;" is more or less the standard way to do it.

1 0

Re: msvcrt: remove hack
by Marcus Meissner 30 May '13

30 May '13

On Wed, May 29, 2013 at 10:23:29PM -0700, Austin English wrote: > This was added 12 years ago in 1db20bfd33f9c1486a1a662c2f78f45d00caf24b > > but clang doesn't like it: > clang -m32 -c -I. -I. -I../../include -I../../include -D__WINESRC__ > -D_MT -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing > -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers > -Wstrict-prototypes -Wtype-limits -Wwrite-strings -Wpointer-arith > -gdwarf-2 -gstrict-dwarf -fno-omit-frame-pointer -g -O2 -o process.o > process.c > process.c:363:10: warning: explicitly assigning a variable of type > 'int' to itself [-Wself-assign] > action = action; /* Remove warning */ > ~~~~~~ ^ ~~~~~~ > 1 warning generated. > > and no modern gcc/other compiler that I can find cares. It was probably added because of a tool that warned of the unused parameter ... So it all goes in circles. ;) Ciao, Marcus

4 6

Re: dinput: Update existing joystick values after setting the range property
by Austin English 30 May '13

30 May '13

On Sun, Apr 14, 2013 at 2:25 PM, Bassi, Gurmail <Gurmail.Bassi(a)mottmac.com> wrote: > ePSXe polls the joystick immediately after setting the axis range property, and uses the returned values as its 'centered' calibrated values - therefore the existing joystick state values must be remapped to the new range (or else the joystick may not have the full range of motion) > > > This patch has been tested on Ubuntu 12.04.1 LTS x86-64, and the behaviour confirmed to match Microsoft's DirectInput implementation on Windows XP 32-bit (Microsoft's DirectInput seems to update the existing joystick values to the new range prior to polling even if the joystick hasn't been moved) > > > --- > dlls/dinput/joystick.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 47 insertions(+) Hi Gurmail, This appears to have caused a regression, see http://bugs.winehq.org/show_bug.cgi?id=33496 (normally I'd cc you on the bug, but you don't seem to have a bugzilla account). -- -Austin

1 0

Re: (try 2)macdrv: rework the way we handle cursor position and composition text
by Ken Thomases 29 May '13

29 May '13

On May 29, 2013, at 1:11 PM, Aric Stewart wrote: > --- > dlls/winemac.drv/cocoa_window.m | 9 +---- > dlls/winemac.drv/event.c | 5 --- > dlls/winemac.drv/ime.c | 85 +++++++++++++---------------------------- > dlls/winemac.drv/macdrv.h | 1 - > dlls/winemac.drv/macdrv_cocoa.h | 6 +-- > 5 files changed, 29 insertions(+), 77 deletions(-) > > > <0014-macdrv-rework-the-way-we-handle-cursor-position-and-co.txt> Looks good to me. -Ken

1 0

Re: macdrv: rework the way we handle cursor position and composition text
by Ken Thomases 29 May '13

29 May '13

On May 29, 2013, at 10:37 AM, Aric Stewart wrote: > --- > dlls/winemac.drv/cocoa_window.m | 9 +---- > dlls/winemac.drv/event.c | 3 -- > dlls/winemac.drv/ime.c | 85 +++++++++++++---------------------------- > dlls/winemac.drv/macdrv.h | 1 - > dlls/winemac.drv/macdrv_cocoa.h | 5 +-- > 5 files changed, 29 insertions(+), 74 deletions(-) I know I reviewed this several times, but I failed to notice until now: you didn't remove the definition of IM_SET_CURSOR_POS in the enum and the entry in the table in dbgstr_event(). -Ken

2 1

GSoC 2013 has begun!
by Austin English 29 May '13

29 May '13

Howdy, Wine has accepted 4 student proposals for Google Summer of Code 2013: George Stephanos, "Registry - implement merging between HKCR and HKCU\Software\Classes" mentored by Detlef Riekenberg Jactry Zeng, "Implement ITextDocument in Richedit" mentored by Huw Davies John Chadwick, "MSXML - Implement XPATH without libxml2" mentored by Nikolay Sivov Zhan Jianyu, "Improve and add missing feature of vbscript component" mentored by Jacek Caban Welcome to Wine and good luck on your projects! As a reminder, here are the next few big dates: May 27th - June 17: community bonding period. Students/mentors should use this time to iron out details of the project, learn the code base and get to know others on the project. June 17: coding officially begins July 29th - Aug 2nd: mid-term reviews are submitted. Mentors/students, please feel free to contact me or André Hentschel if you have any GSoC administrative/procedural questions. Otherwise use wine-devel or ask your mentor for help with learning the Wine development process/codebase/etc. -- -Austin

1 1

Re: [1/3] DWrite: Added missing JUSTIFIED text alignment mode.
by Nikolay Sivov 29 May '13

29 May '13

Hi, Fabian. You forgot to attach patches 2 and 3.

1 0

re: [PATCH (try 2)] dsound: Copy SetFormat format more directly if we're not in PRIMARYWRITE mode
by Maarten Lankhorst 29 May '13

29 May '13

NAK. + HeapFree(GetProcessHeap(), 0, device->primary_pwfx); + device->primary_pwfx = DSOUND_CopyFormat(passed_fmt); Copy to a tmp buffer first and return E_OUTOFMEMORY instead of reassigning primary_pwfx to NULL? ~Maarten

2 1

Re: Christian Costa : d3dx9_36: Implement skin mesh loading.
by gurketsky 29 May '13

29 May '13

Hi, just saw that a bit late. Is there a reason for the rewrite of hr? Could any call return anything other than D3D_OK and succeed (maybe S_FALSE)? It may be possible to drop "if (FAILED(hr)) goto end;" (for both cases) and "hr = D3D_OK"... Also the function (parse_mesh) seems to overwrite the result anyway to D3D_OK on success, so I see no reason to change the hr to D3D_OK in parse_skin_mesh_info. Cheers Rico On 28.05.2013 22:16, Alexandre Julliard wrote: > + if (SUCCEEDED(hr)) > + hr = mesh_data->skin_info->lpVtbl->SetBoneOffsetMatrix(mesh_data->skin_info, index, > + (const D3DMATRIX*)(data + nb_influences * (sizeof(DWORD) + sizeof(FLOAT)))); > + if (FAILED(hr)) > + goto end; > + } > + > + hr = D3D_OK;

2 1

Re: Increment, Decrement, and Assignment Operators
by Alexandre Julliard 29 May '13

29 May '13

Stephen Leibowitz <librestephen(a)gmail.com> writes: > Hello: > > This is my first patch. The changes are minor. If it works out, I may > attempt something more substantial. Doing cosmetic changes like this across the code base is not a good idea. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] gdi32: Ask for a newer FreeType version
by Alexandre Julliard 29 May '13

29 May '13

Detlef Riekenberg <wine.dev(a)web.de> writes: > @@ -3686,7 +3686,8 @@ static BOOL init_freetype(void) > WINE_MESSAGE( > "Wine cannot find the FreeType font library. To enable Wine to\n" > "use TrueType fonts please install a version of FreeType greater than\n" > - "or equal to 2.0.5.\n" > + "or equal to 2.1.4.\n" > + "(Fonts with vertical metrics need at least version 2.4 of FreeType)\n" > "http://www.freetype.org\n"); > return FALSE; > } > @@ -3749,6 +3750,7 @@ sym_not_found: > "Wine cannot find certain functions that it needs inside the FreeType\n" > "font library. To enable Wine to use TrueType fonts please upgrade\n" > "FreeType to at least version 2.1.4.\n" > + "(Fonts with vertical metrics need at least version 2.4 of FreeType)\n" That's not useful. There are probably very few users who still have such an old Freetype, and even if they do, telling them about vertical metrics will only confuse them. -- Alexandre Julliard julliard(a)winehq.org

1 0

re: Increment, Decrement, and Assignment Operators
by Dan Kegel 29 May '13

29 May '13

Watch http://source.winehq.org/patches/ for your patch's status (see legend at bottom). It might be rejected since it doesn't actually fix any behavioral problem in Wine. IIRC trivial cleanups in code are frowned upon, especially if they cross many modules. Better to pick some small real problem and fix it. How about http://bugs.winehq.org/show_bug.cgi?id=32922 ?

1 0

Re: gdi32/tests: Do not try to test OS/2 win metrics on non-windows fonts
by Dmitry Timoshkov 29 May '13

29 May '13

Aric Stewart <aric(a)codeweavers.com> wrote: > + ascent = GET_BE_WORD(tt_os2.usWinAscent); > + descent = GET_BE_WORD(tt_os2.usWinDescent); > + cell_height = ascent + descent; > + ok(ntm->ntmCellHeight == cell_height, "%s: ntmCellHeight %u != %u, os2.usWinAscent/os2.usWinDescent %u/%u\n", > + font_name, ntm->ntmCellHeight, cell_height, ascent, descent); > + > + SetLastError(0xdeadbeef); The SetLastError() call should be left at its previous position. -- Dmitry.

1 0

Re: [PATCH] winhttp: initialize schemes (Coverity) (resubmit)
by Hans Leidekker 28 May '13

28 May '13

On Tue, 2013-05-28 at 22:46 +0200, Marcus Meissner wrote: > 1020943 Uninitialized scalar variable > > schemes is |= in the called function, but not > initialized before. > > Was submitted 2 weeks ago already, I think it is good. It's good but query_auth_schemes is called from more places, so it would be better to fix it in that helper, without touching the parameter on error.

1 0

Re: [PATCH] dsound: Copy SetFormat format more directly if we're not in PRIMARYWRITE mode
by Marvin 28 May '13

28 May '13

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=25741 Your paranoid android. === WXPPROSP2 (32 bit dsound) === dsound.c:1433: Test failed: validbits: 0 === WXPX64 (32 bit dsound) === dsound.c:1433: Test failed: validbits: 0 === W2K3R2SESP2 (32 bit dsound) === dsound.c:1433: Test failed: validbits: 0 === WXPX64 (64 bit dsound) === dsound.c:1433: Test failed: validbits: 0

1 0

Re: [5/5] kernel32: Don't move fixed memory area in GlobalReAlloc. (try 2)
by Alexandre Julliard 28 May '13

28 May '13

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > This closes bug 14819. > > try 2: > * tests/heap.c was updated to avoid dependency on heap reallocation > strategies. > * Use HeapAlloc instead of LocalAlloc in some places. In those cases, > they want large memory blocks, not pinned memory addresses. They are using LMEM_MOVEABLE though. A few more tests for this flag would probably be a good idea. -- Alexandre Julliard julliard(a)winehq.org

1 0

dxgi, directx 11 and code duplication
by John Yani 28 May '13

28 May '13

Hi. I'm trying to set up a minimal implementation that will allow to run Assasin's Creed III. It uses these 2 functions: dxgi.dll.CreateDXGIFactory1 d3d11.dll.D3D11CreateDevice See http://bugs.winehq.org/show_bug.cgi?id=24517 I need an advice on how to implement CreateDXGIFactory1 without duplicating a lot of code. There are problems with a straight-forward implementation: interface IWineDXGIFactory : IDXGIFactory { struct wined3d *get_wined3d(); } interface IWineDXGIFactory1 : IDXGIFactory1 { struct wined3d *get_wined3d(); } The problem is how to reuse functions where IWineDXGIFactory is used. Obviously we can't pass IWineDXGIFactory1 object where IWineDXGIFactory is required. Because the next time we call get_wined3d, a crash will occur (a different function will be called). Is there a more elegant solution than duplicating code and replacing all occurencdes of IWineDXGIFactory by IWineDXGIFactory1? Btw, D3D11CreateDevice accepts both IDXGIFactory and IDXGIFactory1: http://msdn.microsoft.com/en-us/library/windows/desktop/ff476082(v=vs.85).a…

2 1

dxgi, directx 11 and code duplication
by John Yani 27 May '13

27 May '13

Hi. I'm trying to set up a minimal implementation that will allow to run Assasin's Creed III. It uses these 2 functions: dxgi.dll.CreateDXGIFactory1 d3d11.dll.D3D11CreateDevice See http://bugs.winehq.org/show_bug.cgi?id=24517 I need an advice on how to implement CreateDXGIFactory1 without duplicating a lot of code. There are problems with a straight-forward implementation: interface IWineDXGIFactory : IDXGIFactory { struct wined3d *get_wined3d(); } interface IWineDXGIFactory1 : IDXGIFactory1 { struct wined3d *get_wined3d(); } The problem is how to reuse functions where IWineDXGIFactory is used. Obviously we can't pass IWineDXGIFactory1 object where IWineDXGIFactory is required. Because the next time we call get_wined3d, a crash will occur (a different function will be called). Is there a more elegant solution than duplicating code and replacing all occurencdes of IWineDXGIFactory by IWineDXGIFactory1? Btw, D3D11CreateDevice accepts both IDXGIFactory and IDXGIFactory1: http://msdn.microsoft.com/en-us/library/windows/desktop/ff476082(v=vs.85).a…

1 0

Release plans
by Alexandre Julliard 27 May '13

27 May '13

Folks, Now that the Mac driver is fully merged and all the bugs fixed (right?), it's time to think about the next stable release. I would like to start the code freeze after the next release cycle, so two weeks from now. Hopefully, like last time we can keep the freeze period relatively short, so that it won't impact the Summer of Code projects too much. Note that in general, new test cases can be accepted during code freeze, and it's always a good idea to start a GSoC project by writing tests. On a related note, the test results on test.winehq.org haven't been looking very good lately; in particular, we need to make the full test suite succeed on the new testbot VMs, since the old ones are being phased out. Fixes to make tests succeed on Windows can go in during code freeze, since by definition they don't impact the Wine code, so this would be a good opportunity to try to improve the situation. OTOH the regression status is looking good, I'm very pleased with how we've kept the regression count under control during the 1.5 series. Hopefully we can even fix a few more during code freeze and set a lower baseline for the next development series. Thanks all for your help, and let's make this another great release! -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: winmm: load winealsa if winepulse is found
by cfsworks＠gmail.com 27 May '13

27 May '13

Wouldn't this cause applications which use winmm for waveout to be diverted to ALSA as well? Maybe both drivers should be loaded side by side, with winepulse handling everything but MIDI.

1 0

Re: [PATCH v23] winepulse: add pulseaudio driver
by Marvin 27 May '13

27 May '13

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=25724 Your paranoid android. === WVISTAX64 (32 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 00000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 00000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 00000000 00000000 === W2K8SE (32 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 00000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 00000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 00000000 00000000 === W7PROX64 (32 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 00000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 00000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 88890008 00000000 === TEST64_W7SP1 (32 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 00000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 00000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 00000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 88890008 00000000 === WVISTAX64 (64 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 0000000000000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 0000000000000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 00000000 0000000000000000 === W7PROX64 (64 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 0000000000000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 0000000000000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 88890008 0000000000000000 === TEST64_W7SP1 (64 bit render) === render.c:520: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:527: Test failed: Shared IsFormatSupported with nAvgBytesPerSec=0 and nBlockAlign=0 returned 88890008 0000000000000000 render.c:586: Test failed: Shared IsFormatSupported with wValidBitsPerSample=0 returned 88890008 0000000000000000 render.c:599: Test failed: Shared IsFormatSupported with wValidBitsPerSample += 1 returned 88890008 0000000000000000 render.c:616: Test failed: Shared IsFormatSupported with wBitsPerSample=256 returned 88890008 0000000000000000

1 0

Making SetThreadPriorityBoost claim success
by Dan Kegel 27 May '13

27 May '13

http://bugs.winehq.org/show_bug.cgi?id=22280 just got another dup. How awful would it be to make SetThreadPriorityBoost just return success, as in the patch attached to http://bugs.winehq.org/show_bug.cgi?id=22280#c1 http://bugs.winehq.org/attachment.cgi?id=27224&action=diff&context=patch&co… ? We'd have to mark one more test TODO_WINE. Related if catty discussion: http://www.codinghorror.com/blog/2006/08/thread-priorities-are-evil.html

2 1

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-devel