- Wine-Devel - list.winehq.org

OXID marshaling error
by Bill Medland Jan. 11, 2005

Jan. 11, 2005

Any idea what is going wrong in the OXID handling here? Is "a" the first thread of the new process? Is the problem that it should be "b"? (I've removed a few extraneous traces) (This is after the COM_CreateApartment fixes patch) 0009:trace:ole:create_marshalled_proxy rclsid={4dfceca2-156a-4e02-9d02-2e5253ad1d01}, iid={00000001-0000-0000-c000-000000000046} 0009:trace:ole:create_marshalled_proxy waiting for \\.\pipe\{4DFCECA2-156A-4E02-9D02-2E5253AD1D01} 0009:trace:ole:create_server activating local server 'L"G:\\runtime\\a4wlpmgr.exe -Embedding"' for {4DFCECA2-156A-4E02-9D02-2E5253AD1D01} 0009:trace:ole:create_marshalled_proxy waiting for \\.\pipe\{4DFCECA2-156A-4E02-9D02-2E5253AD1D01} 0009:warn:ole:create_marshalled_proxy Could not open named pipe to broker \\.\pipe\{4DFCECA2-156A-4E02-9D02-2E5253AD1D01}, le is 2 000b:trace:ole:DllMain 0x540000 0x1 0x1 0009:trace:ole:create_marshalled_proxy waiting for \\.\pipe\{4DFCECA2-156A-4E02-9D02-2E5253AD1D01} 0009:warn:ole:create_marshalled_proxy Could not open named pipe to broker \\.\pipe\{4DFCECA2-156A-4E02-9D02-2E5253AD1D01}, le is 2 000b:trace:ole:CoInitializeEx ((nil), 0) 000b:trace:ole:CoInitializeEx () - Initializing the COM libraries 000b:trace:ole:RunningObjectTableImpl_Initialize () 000b:trace:ole:COM_CreateApartment thread 0xb is entering the multithreaded apartment 000b:trace:ole:CoRegisterClassObject ({4dfceca2-156a-4e02-9d02-2e5253ad1d01},0x405701a0,0x00000004,0x00000005,0x412054) 000b:trace:ole:CoMarshalInterface (0x4006b7a0, {00000001-0000-0000-c000-000000000046}, 0x405701a0, 0, (nil), 1) 000b:trace:ole:CoGetStandardMarshal ({00000001-0000-0000-c000-000000000046},0x405701a0,0,(nil),1,0x51fd84) 000b:trace:ole:CoMarshalInterface Using standard marshaling 000b:trace:ole:CoMarshalInterface Calling IMarshal::MarshalInterace 000b:trace:ole:StdMarshalImpl_MarshalInterface (...,{00000001-0000-0000-c000-000000000046},...) 000b:trace:ole:start_apartment_listener_thread apt->listenertid=0 000d:trace:ole:apartment_listener_thread Apartment listener thread starting on (\\.\pipe\WINE_OLE_StubMgr_0000000a00000000) 000b:trace:ole:CoGetPSClsid () riid={00000001-0000-0000-c000-000000000046}, pclsid=0x51fd18 000b:trace:ole:CoGetPSClsid () Returning CLSID={00000320-0000-0000-c000-000000000046} 000b:trace:ole:CoGetClassObject CLSID: {00000320-0000-0000-c000-000000000046}, IID: {d5f569d0-593b-101a-b569-08002b2dbf7a} 000b:trace:ole:COMPOBJ_DLLList_Add 000b:trace:ole:PSFacBuf_CreateStub ({00000001-0000-0000-c000-000000000046},0x405701a0,0x51fd10) 000b:fixme:ole:StdMarshalImpl_MarshalInterface table marshalling unimplemented 000b:warn:ole:get_stub_manager_from_object Could not map OXID a00000000 to apartment object 000b:trace:ole:register_ifstub constructing new stub manager 000b:warn:ole:register_ifstub Could not map OXID a00000000 to apartment object -- Bill Medland mailto:billmedland(a)mercuryspeed.com http://webhome.idirect.com/~kbmed

2 2

Re: USER32: cursoricon
by Alexandre Julliard Jan. 11, 2005

Jan. 11, 2005

Vitaly Lipatov <lav(a)etersoft.ru> writes: > Changelog: > return fail if file format is unknown This will make it fail for all normal cursor/icon files. These don't have a header so you can't check a magic number. -- Alexandre Julliard julliard(a)winehq.org

2 2

ole32: questions about the COINIT flags
by Paul Vriens Jan. 11, 2005

Jan. 11, 2005

Hi, I was looking through some MSDN doc: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/com/htm/cm… there it's stated that: If neither concurrency model is specified by the dwCoInit parameter, the default is COINIT_MULTITHREADED. Does this mean we should change the way we interpret these flags (in compobj.c for example) because COINIT_MULTITHREADED equals 0 (which is a bit strange value for a flag)? The native QUARTZ.DDL calls CoInitializeEx with a COINIT of 4 which effectively has the COINIT_DISABLE_OLE1DDE and the COINIT_MULTITHREADED flags set. Shouldn't we have a check for COINIT_APARTMENTTHREADED and if this flag is not set we have to assume COINIT_MULTITHREADED? Especially because both these flags cannot be set at the same time (see mentioned MSDN doc). Cheers, Paul.

2 1

Re: [QUARTZ] Get rid of a CLSID fixme
by Christian Costa Jan. 11, 2005

Jan. 11, 2005

> Message du 11/01/05 13:50 > De : "Paul Vriens" > A : "wine-patches(a)winehq.org" > Copie à : "Christian Costa" > Objet : [QUARTZ] Get rid of a CLSID fixme > Hi, > > just needed an extra CLSID in the registry to avoid some fixme's: > > fixme:ole:CoCreateInstance no classfactory created for CLSID > {060af76c-68dd-11d0-8fc1-00c04fd9189d} As long as this object is not implemented, the FIXME should remain to remind us that something is missing from quartz. Bye, Christian

1 0

wine alsa detection
by Joris Huizer Jan. 11, 2005

Jan. 11, 2005

Hello, Today I switched back to alsa 0.9 (downgraded everything that had to do with alsa 1.0 as far as I can tell) but still, alsa is detected as available: checking alsa/asoundlib.h usability... yes checking alsa/asoundlib.h presence... yes checking for alsa/asoundlib.h... yes checking for snd_pcm_open in -lasound... yes And wine breaks when compiling wine alsa - then it does all the compiling again as dependencies are assumed to be wrong -- finally breaking again at the same place; Are you sure your configuration patch tests for those failing functions? I attached a log of it failing with wine 0.9 - maybe you have to test for some (all??) of those? If you have reason to think I forgot something when downgrading, please tell me what it was (I'm using debian/stable - with added unofficial alsa packages, of http://afavant.elte.hu/~wferi/lilypond/, as suggested earlier on the list ) regards, Joris

2 3

Re: Janitorial : Use Compiler Warnings ( -Wmissing-declarations ) ( #1, take 2 )
by Alexandre Julliard Jan. 11, 2005

Jan. 11, 2005

Rémi Assailly <remi.assailly(a)free.fr> writes: > diff -u libs/wine/debug.c libs/wine/debug.c > --- libs/wine/debug.c 2005-01-10 20:12:34.748897920 +0100 > +++ libs/wine/debug.c 2005-01-10 19:18:26.000000000 +0100 > @@ -61,6 +61,9 @@ > return strcmp( name, *chan + 1 ); > } > > +void *__wine_dbg_register( char * const *channels, int nb ); > +void __wine_dbg_unregister( void *channel ); This is useless, please don't add such code only to quiet optional warnings. > diff -u include/wine/unicode.h include/wine/unicode.h > --- include/wine/unicode.h 2005-01-09 12:09:03.000000000 +0100 > +++ include/wine/unicode.h 2005-01-10 19:23:51.000000000 +0100 > @@ -95,6 +95,8 @@ > extern int vsprintfW( WCHAR *str, const WCHAR *format, va_list valist ); > extern int vsnprintfW( WCHAR *str, size_t len, const WCHAR *format, va_list valist ); > > +extern int get_decomposition(WCHAR src, WCHAR *dst, unsigned int dstlen); > + get_decomposition is an internal function, it shouldn't be exported in the public header. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: Minor wine-devel DCOM corrections
by Dan Kegel Jan. 11, 2005

Jan. 11, 2005

Rob Shearman wrote: > We have a number of marshaling tests in our test framework at the moment > (in wine/dlls/ole32/tests/marshal.c). It is hard to do anything > cross-process (or cross-machine!) because of the way the test framework > is done. Mike did suggest using environment variables to hack around > those limitations, but so far that hasn't turned into any code from > either of us. Hey, that's not a bad idea. You could set an environment variable, then rerun the same program with the same arguments, and your test could sense the environment variable and run the 2nd process's test role. Is there a generic win32 way to run yourself again with the same arguments? If not, it'd probably only take a little tweaking of include/wine/test.h to set one up... - Dan -- Trying to get a job as a c++ developer? See http://kegel.com/academy/getting-hired.html

2 1

Re: Correction to COINIT_MULTITHREADED test
by Paul Vriens Jan. 11, 2005

Jan. 11, 2005

On Mon, 2005-01-10 at 21:20, Bill Medland wrote: > Bill Medland (billmedland(a)mercuryspeed.com) > Correct testing for multithreaded > (as spotted by Paul Vriens; fix suggested by Robert Shearman) > Hi Bill, it wasn't the check that was way over my head. The thing is, that with this code change we walk a new path in COM_CreateApartment. I don't know what the result is but in CoInitializeEx we do: 545 if (!(apt = COM_CurrentInfo()->apt)) 546 { 547 apt = COM_CreateApartment(dwCoInit); 548 if (!apt) return E_OUTOFMEMORY; 549 } and apparently that leaves us with a E_OUTOFMEMORY all the time. Which means that we will not come to the part: 559 COM_CurrentInfo()->inits++; and that leads to errors as: trace:ole:CoInitializeEx ((nil), 4) trace:ole:COM_CreateApartment thread 0xd is entering the multithreaded apartment trace:ole:CoInitializeEx ((nil), 2) err:ole:CoInitializeEx Attempt to change threading model of this apartment from 0x0 to 0x2 which also shows that the check is wrong. When CoInitializeEx is called with COINIT_DISABLE_OLE1DDE it shouldn't go in. What MSDN also says is: Any combination of values from the COINIT enumeration can be used, except that the COINIT_APARTMENTTHREADED and COINIT_MULTITHREADED flags cannot both be set. So should that be accounted for as well? Cheers, Paul.

1 0

comctl32.dll and commctrl.dll
by Dripple Jan. 11, 2005

Jan. 11, 2005

Hi all, I'm a newbie in wine using and wine developpement. I just need information about comctl32.dll and commctrl.dll. I'm trying to have this app (http://appdb.winehq.org/appview.php?appId=1485&versionId=2451) working. In fact, it works. But only using native comctl32.dll, which requires native commctrl.dll. Can anybody explain me what's missing in buitlin comctl32.dll and/or commctrl.dll ? Is it planned to fix commctrl.dll, to avoid piracy ? Thanks. Dripple

5 7

Re: advpack: add two missing stubs
by Robert Reif Jan. 10, 2005

Jan. 10, 2005

Rob Shearman wrote: > > I think you forgot to include the tests. > > Rob > Past my bed time. diff -u -N wine.cvs/dlls/advpack/tests/advpack.c wine/dlls/advpack/tests/advpack.c --- wine.cvs/dlls/advpack/tests/advpack.c 1969-12-31 19:00:00.000000000 -0500 +++ wine/dlls/advpack/tests/advpack.c 2005-01-10 22:54:50.000000000 -0500 @@ -0,0 +1,66 @@ +/* + * Unit tests for advpack.dll + * + * Copyright (c) 2005 Robert Reif + * + * 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#define NONAMELESSSTRUCT +#define NONAMELESSUNION +#include <windows.h> + +#include "wine/test.h" +#include "advpub.h" + +static void version_test() +{ + HRESULT hr; + DWORD major, minor; + + major = minor = 0; + hr = GetVersionFromFile("kernel32.dll", &major, &minor, FALSE); + ok (hr == S_OK, "GetVersionFromFileEx(kernel32.dll) failed, returned " + "0x%08lx\n", hr); + + trace("kernel32.dll Language ID: 0x%08lx, Codepage ID: 0x%08lx\n", + major, minor); + + major = minor = 0; + hr = GetVersionFromFile("kernel32.dll", &major, &minor, TRUE); + ok (hr == S_OK, "GetVersionFromFileEx(kernel32.dll) failed, returned " + "0x%08lx\n", hr); + + trace("kernel32.dll version: %d.%d.%d.%d\n", HIWORD(major), LOWORD(major), + HIWORD(minor), LOWORD(minor)); +} + +static void admin_test() +{ + BOOL res; + DWORD reserved; + + res = IsNTAdmin(0, &reserved); + ok (res == TRUE || res == FALSE, "IsNTAdmin returned: 0x%08x\n", res); + + trace("IsNTAdmin = %s\n", res == TRUE ? "TRUE" : res == FALSE ? "FALSE" : + "???"); +} + +START_TEST(advpack) +{ + version_test(); + admin_test(); +} diff -u -N wine.cvs/dlls/advpack/tests/.cvsignore wine/dlls/advpack/tests/.cvsignore --- wine.cvs/dlls/advpack/tests/.cvsignore 1969-12-31 19:00:00.000000000 -0500 +++ wine/dlls/advpack/tests/.cvsignore 2005-01-10 21:17:35.000000000 -0500 @@ -0,0 +1,3 @@ +Makefile +advpack.ok +testlist.c diff -u -N wine.cvs/dlls/advpack/tests/Makefile.in wine/dlls/advpack/tests/Makefile.in --- wine.cvs/dlls/advpack/tests/Makefile.in 1969-12-31 19:00:00.000000000 -0500 +++ wine/dlls/advpack/tests/Makefile.in 2005-01-10 21:07:22.000000000 -0500 @@ -0,0 +1,13 @@ +TOPSRCDIR = @top_srcdir@ +TOPOBJDIR = ../../.. +SRCDIR = @srcdir@ +VPATH = @srcdir@ +TESTDLL = advpack.dll +IMPORTS = advpack user32 kernel32 version + +CTESTS = \ + advpack.c + +(a)MAKE_TEST_RULES@ + +### Dependencies:

1 0