February 2008 - Wine-Devel - list.winehq.org

Help me(Ajudem-me)C compiler cannot create executables
by flavio melo Feb. 26, 2008

Feb. 26, 2008

Hello I'm Flavio Melo(brazilian)! Well, i use the ubuntu 7.04, and there is any time that i try use the wine,but i no get install it, i can't for net because i haven't internet (repository), but i download the file wine-0.9.54 of your site in the other computer(in my job), and, i follow the instruccions(i think there are corrects, get it read manuals) and my terminal said: ---------------------------------------------------------------------------- flavio(a)Discipulo:~$ cd wine flavio(a)Discipulo:~/wine$ cd tools flavio(a)Discipulo:~/wine/tools$ ./wineinstall WINE Installer v0.75 ~/wine ~/wine/tools Running configure... configure: creating cache config.cache checking build system type... i686-pc-linux-gnulibc1 checking host system type... i686-pc-linux-gnulibc1 checking whether make sets $(MAKE)... yes checking for gcc... gcc checking for C compiler default output file name... configure: error: C compiler cannot create executables See `config.log' for more details. Configure failed, aborting install. flavio(a)Discipulo:~/wine/tools$ ----------------------------------------------------------------------------- I wrote to you for know if you can help me(Please), what i must make for my gcc compile? Thank for attencion! Meu testemunho... http://www.semearcristo.net/art/art_00007.aspx?idPag=1&idReg=1&idCat=6&strB… --------------------------------- Abra sua conta no Yahoo! Mail, o único sem limite de espaço para armazenamento!

3 2

Re: comctl32: rebar[1/4]: an implementation of RB_SIZETORECT (fixes bug #7774)
by John Klehm Feb. 26, 2008

Feb. 26, 2008

Missing patch On Tue, Feb 26, 2008 at 3:36 PM, Mikołaj Zalewski <mikolaj(a)zalewski.pl> wrote: > I couldn't work out the algorithm to make it pixel-for-pixel identical > as on Windows but this function implements the general idea that I hope > will work for applications. > > >

1 0

re: gdi32: Protect against short sized LOGPALETTE in CreatePalette.
by Dan Kegel Feb. 26, 2008

Feb. 26, 2008

Hi, your patch uses IsBadReadPtr(), but that function is deprecated in favor of using an exception handler. See http://www.winehq.org/pipermail/wine-devel/2006-April/046560.html Here's an example (sorry it's a messy one, it's just the first one google found): http://www.winehq.org/pipermail/wine-patches/2004-November/013791.html Please try recoding using that approach. If you have trouble getting this patch in, you might want to file a bug at http://bugs.winehq.org for your problem and link to your patch; that will record the problem so somebody else can pick it up later. - Dan

3 2

re: gdi32: Patch review request
by Dan Kegel Feb. 26, 2008

Feb. 26, 2008

Daniel Mario Vega wrote: >A few days ago I sent 2 patches for gdi32 to wine-patches >that passed unnoticed. I would like if someone could give > me some feedback on them. OK. I replied to each of them just now with my guess. Don't worry, patches being ignored is normal, it just means you have to improve them if possible and resend. If you do resend them, be sure to give the same subject as last time, with '(resend)' appended if no changes made, or '(try 2)' appended if there were changes, and append a note about the changes to the description.

1 0

re: gdi32: StartDocA support for legacy apps with incomplete DOCINFOA parameter.
by Dan Kegel Feb. 26, 2008

Feb. 26, 2008

This patch looks nice, but it could use a conformance test. Please add one and resend. And consider filing a bug and referring to the bug in the patch description, that never hurts. - Dan

1 0

Re: [try4] user32: cursoricon.c. Fix 16b BMP handling with BT_BITFIELD.
by Dmitry Timoshkov Feb. 26, 2008

Feb. 26, 2008

"Alexander Demakin" <avd(a)etersoft.ru> wrote: > This patch fixes image header handling with BT_BITFIELD value in biCompression > field. 3 DWORDs of mask were included into image bits, it caused some color > troubles and image horizontal shift. Size of bitmap info structure includes > now this bytes. > Changelog: > user32.dll Fix 16b BMP handling with BT_BITFIELD. I wonder whether gcc complained loudly that 'result' may be used uninitialized. -- Dmitry.

1 1

marshalled ITypeComp
by Guillermo Winkler Feb. 26, 2008

Feb. 26, 2008

Hi, I'm having some problems regarding ITypeInfo/ITypeComp implementation. IDispatch* pdisp; ITypeInfo* ptinfo; HRESULT hr = pdisp->GetTypeInfo(0, LOCALE_SYSTEM_DEFAULT, &typeinfo); hr = ptinfo->GetTypeComp(&ptcomp); pdisp is created from hr = CoCreateInstance(clsid, NULL, CLSCTX_SERVER, IID_IDispatch, (void**)&pdisp); That is, the server is an outprocess exe. In this case marshalling of ITypeComp fails in first instance because ITypeComp_fnQueryInterface (oleaut32\typelib.c) resolves on -> ITypeInfo_QueryInterface (oleaut32\typelib.c) That is not considering IID_ITypeComp as a valid alternative(but it is) if(IsEqualIID(riid, &IID_IUnknown) || IsEqualIID(riid,&IID_ITypeInfo)|| IsEqualIID(riid,&IID_ITypeInfo2)) *ppvObject = This; When I fixed this, I have a different problem now: fixme:ole:DllGetClassObject CLSID: {00020425-0000-0000-C000-000000000046} IID: {D5F569D0-593B-101A-B569-08002B2DBF7A} That is PSTypeComp to be marshalled asking for factory buffer IID_IPSFactoryBuffer. I would like to follow up with someone (if available) just to be sure I'm not wasting time on the wrong track. Thanks, Guillermo

2 2

gdi32: Patch review request
by Daniel Mario Vega Feb. 26, 2008

Feb. 26, 2008

Hi! A few days ago I sent 2 patches for gdi32 to wine-patches that passed unnoticed. I would like if someone could give me some feedback on them. Let me first give you some background about these patches. There is an application called "Compumap" that is a street map of some Argentine cities. It is rather popular here because some years ago it was distributed with a newspaper. The program works quite well under Wine (although the installer still has some problems), but when you try to print a map it crashes. A few weeks ago I had some free time and I decided to try to find out why it crashes. I run it through the Wine Debugger and also under a debugger in Windows XP. The problem happened to be two API calls with incomplete/invalid parameters. Windows XP returned NULL or ignore the wrong data, while Wine crashes. 1) gdi32: Protect against short sized LOGPALETTE in CreatePalette. http://www.winehq.org/pipermail/wine-patches/2008-February/050354.html This crash was caused by "CreatePalette(HDC, LOGPALETTE)" being called with a short allocated palette. Windows XP returned NULL in that case. I tried to reproduce that behavior by first checking the parameter with "IsBadReadPtr()" function. 2) gdi32: StartDocA support for legacy apps with incomplete DOCINFOA parameter. http://www.winehq.org/pipermail/wine-patches/2008-February/050357.html This issue was a call to "StartDocA(HDC, DOCINFOA)" with structure DOCINFOA not completely initialized (there is garbage in the last 2 fields). I debugged this API and found that Windows XP ignored those fields when the data "seems to be invalid". I think that the last 2 fields were introduced in Win32 and some applications ported from Win16 "forget" to clear them. I simulated Windows XP behaviour by putting some safe checks on those fields. With those 2 fixes, the application is able to print under Wine. Both patches are short and rather easy to review. I will appreciate if someone could give me some feedback on them. Thanks ... -- Daniel Mario Vega

1 0

Re: opengl32/tests: Add the trailing '\n' to an ok() call and make a spelling fix.
by Paul Vriens Feb. 26, 2008

Feb. 26, 2008

Francois Gouget wrote: > --- > dlls/opengl32/tests/opengl.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c > index 3836ba7..344e2c6 100644 > --- a/dlls/opengl32/tests/opengl.c > +++ b/dlls/opengl32/tests/opengl.c > @@ -384,9 +384,9 @@ START_TEST(opengl) > > /* We shouldn't be able to create a context from a hdc which doesn't have a pixel format set */ > hglrc = wglCreateContext(hdc); > - ok(hglrc == NULL, "wglCreateContext should fail when no pixel format has been set, but it passed"); > + ok(hglrc == NULL, "wglCreateContext should fail when no pixel format has been set, but it passed\n"); > error = GetLastError(); > - ok(error == ERROR_INVALID_PIXEL_FORMAT, "expected ERROR_INVALID_PIXEL_FORMAT for wglCreateContext without a pixelformat set, but recevied %#x", error); > + ok(error == ERROR_INVALID_PIXEL_FORMAT, "expected ERROR_INVALID_PIXEL_FORMAT for wglCreateContext without a pixelformat set, but recieved %#x\n", error); So where is the spelling fix ;-) Shouldn't that be "received"? -- Cheers, Paul.

1 0

Re: d3dx9: Add IMPORTLIB to every Makefile.in [PATCH 4/9]
by Alexandre Julliard Feb. 26, 2008

Feb. 26, 2008

tony.wasserka(a)freenet.de writes: >>From 881b8de1e044ff90bcb543501208a9cd879cbc16 Mon Sep 17 00:00:00 2001 > From: Tony Wasserka <tony.wasserka(a)freenet.de> > Date: Sat, 23 Feb 2008 13:55:40 +0100 > Subject: [PATCH] d3dx9: Add IMPORTLIB to every Makefile.in That shouldn't be needed, there are no such import libs on Windows. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: d3dx9: Implement D3DXCheckVersion [PATCH 1/9]
by Alexandre Julliard Feb. 26, 2008

Feb. 26, 2008

tony.wasserka(a)freenet.de writes: > D3DXCheckVersion doesn't check for one specific D3DX version, but > for all that are compatible with our Wine code (i.e. d3dx9_24 to d3dx9_36) > The reasons for that were explained in these mails by me: > http://www.winehq.org/pipermail/wine-devel/2008-February/062290.html > and http://www.winehq.org/pipermail/wine-devel/2008-February/062292.html This is wrong, you should have a separate version check for each dll that checks the exact version of that dll. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: msvcrt: Implemented wcscpy_s (try 2)
by Alistair Leslie-Hughes Feb. 26, 2008

Feb. 26, 2008

Hi, Is there any thing wrong with this patch? Best Regards Alistair Leslie-Hughes "Alistair Leslie-Hughes" <leslie_alistair(a)hotmail.com> wrote in message news:fpljui$nsa$1(a)ger.gmane.org... > Hi, > Correct msvcrt.spec file > > Changelog: > msvcrt: Implemented wcscpy_s > -------------------------------------------------------------------------------- > From 6860771cc3fc06b3da63f7c674daf71861593ef3 Mon Sep 17 00:00:00 2001 > From: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> > Date: Fri, 22 Feb 2008 13:48:01 +1100 > Subject: [PATCH] Implemented wcscpy_s > > --- > dlls/msvcrt/msvcrt.spec | 1 + > dlls/msvcrt/tests/string.c | 45 > ++++++++++++++++++++++++++++++++++++++++++++ > dlls/msvcrt/wcs.c | 45 > ++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 91 insertions(+), 0 deletions(-) > > diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec > index d6cffcb..7b821e6 100644 > --- a/dlls/msvcrt/msvcrt.spec > +++ b/dlls/msvcrt/msvcrt.spec > @@ -761,6 +761,7 @@ > @ cdecl wcscmp(wstr wstr) ntdll.wcscmp > @ cdecl wcscoll(wstr wstr) MSVCRT_wcscoll > @ cdecl wcscpy(ptr wstr) ntdll.wcscpy > +@ cdecl wcscpy_s(ptr long wstr) MSVCRT_wcscpy_s > @ cdecl wcscspn(wstr wstr) ntdll.wcscspn > @ cdecl wcsftime(ptr long wstr ptr) MSVCRT_wcsftime > @ cdecl wcslen(wstr) ntdll.wcslen > diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c > index c6dcd34..b09d204 100644 > --- a/dlls/msvcrt/tests/string.c > +++ b/dlls/msvcrt/tests/string.c > @@ -50,6 +50,7 @@ static int* (*pmemcmp)(void *, const void *, size_t n); > static int (*pstrcpy_s)(char *dst, size_t len, const char *src); > static int (*pstrcat_s)(char *dst, size_t len, const char *src); > static int (*p_mbsnbcpy_s)(unsigned char * dst, size_t size, const > unsigned char * src, size_t count); > +static int (*p_wcscpy_s)(wchar_t *wcDest, size_t size, const wchar_t > *wcSrc); > > #define SETNOFAIL(x,y) x = (void*)GetProcAddress(hMsvcrt,y) > #define SET(x,y) SETNOFAIL(x,y); ok(x != NULL, "Export '%s' not found\n", > y) > @@ -562,6 +563,47 @@ static void test__mbsnbcpy_s(void) > dest[0], dest[1], dest[2], dest[3], dest[4], dest[5], dest[6], > dest[7]); > } > > +static void test_wcscpy_s(void) > +{ > + static const WCHAR szLongText[] = { > 'T','h','i','s','A','L','o','n','g','s','t','r','i','n','g',0 }; > + static WCHAR szDest[18]; > + static WCHAR szDestShort[8]; > + int ret; > + > + if(!p_wcscpy_s) > + { > + skip("wcscpy_s not found\n"); > + return; > + } > + > + /* Test NULL Dest */ > + ret = p_wcscpy_s(NULL, 16, szLongText); > + ok(ret == EINVAL, "p_wcscpy_s expect EINVAL got %d\n", ret); > + > + /* Test NULL Source */ > + szDest[0] = 'A'; > + ret = p_wcscpy_s(szDest, 16, NULL); > + ok(ret == EINVAL, "expected EINVAL got %d\n", ret); > + ok(szDest[0] == 0, "szDest[0] not 0\n"); > + > + /* Test invalid size */ > + szDest[0] = 'A'; > + ret = p_wcscpy_s(szDest, 0, szLongText); > + ok(ret == EINVAL, "expected ERANGE got %d\n", ret); > + ok(szDest[0] == 0, "szDest[0] not 0\n"); > + > + /* Copy same buffer size */ > + ret = p_wcscpy_s(szDest, 18, szLongText); > + ok(ret == 0, "expected 0 got %d\n", ret); > + ok(lstrcmpW(szDest, szLongText) == 0, "szDest != szLongText\n"); > + > + /* Copy smaller buffer size */ > + szDest[0] = 'A'; > + ret = p_wcscpy_s(szDestShort, 8, szLongText); > + ok(ret == ERANGE, "expected EINVAL got %d\n", ret); > + ok(szDestShort[0] == 0, "szDest[0] not 0\n"); > +} > + > START_TEST(string) > { > char mem[100]; > @@ -577,6 +619,7 @@ START_TEST(string) > SET(pstrcpy_s,"strcpy_s"); > SET(pstrcat_s,"strcat_s"); > SET(p_mbsnbcpy_s,"_mbsnbcpy_s"); > + SET(p_wcscpy_s,"wcscpy_s"); > > /* MSVCRT memcpy behaves like memmove for overlapping moves, > MFC42 CString::Insert seems to rely on that behaviour */ > @@ -599,4 +642,6 @@ START_TEST(string) > test_strcpy_s(); > test_strcat_s(); > test__mbsnbcpy_s(); > + > + test_wcscpy_s(); > } > diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c > index 92dd644..66b8907 100644 > --- a/dlls/msvcrt/wcs.c > +++ b/dlls/msvcrt/wcs.c > @@ -27,6 +27,8 @@ > #include "wine/unicode.h" > #include "wine/debug.h" > > +#include <msvcrt/errno.h> > + > WINE_DEFAULT_DEBUG_CHANNEL(msvcrt); > > > @@ -971,3 +973,46 @@ INT CDECL MSVCRT_iswxdigit( MSVCRT_wchar_t wc ) > { > return isxdigitW( wc ); > } > + > +/********************************************************************* > + * wcscpy_s (MSVCRT.@) > + */ > +INT CDECL MSVCRT_wcscpy_s( MSVCRT_wchar_t* wcDest, size_t numElement, > const MSVCRT_wchar_t *wcSrc) > +{ > + int size = 0; > + > + if(!wcDest) > + return EINVAL; > + > + if(!wcSrc) > + { > + wcDest[0] = 0; > + return EINVAL; > + } > + > + if(numElement <= 0) > + { > + wcDest[0] = 0; > + > + if(numElement < 0) > + return ERANGE; > + else > + return EINVAL; > + } > + > + size = strlenW(wcSrc) + 1; > + > + if(size > numElement) > + { > + wcDest[0] = 0; > + return ERANGE; > + } > + > + if(size > numElement) > + size = numElement; > + > + memcpy( wcDest, wcSrc, size*sizeof(WCHAR) ); > + > + return 0; > +} > + > -- > 1.5.4.1 > > -------------------------------------------------------------------------------- > >

2 1

Re: [6/6] msxml3: Tests for put_data
by Alistair Leslie-Hughes Feb. 26, 2008

Feb. 26, 2008

"Alistair Leslie-Hughes" <leslie_alistair(a)hotmail.com> wrote in message news:fpu31c$upv$4(a)ger.gmane.org... > +#if 0 > +/* For some Reason, IXMLDOMComment_put_data cannot be found even though > it can > + be exists in msxml2.h. > + */ > + /* put data Tests */ > + hr = IXMLDOMComment_put_data(pComment, _bstr_("This &is a > ; test <>\\")); > + ok(hr == S_OK, "ret %08x\n", hr ); > + > +#endif Hi, Im a little confused, IXMLDOMComment_put_data wasnt compiling on my machine, and yet Alexandre changed to include this code and now it compiles correctly? I event tried a >make clean; ./configure; make depend;make which didnt work either. Can anybody explain what was fixed? Best Regards Alistair Leslie-Hughes

2 1

Re: wininet: Fix retrieving the raw headers
by Alexandre Julliard Feb. 26, 2008

Feb. 26, 2008

"Maarten Lankhorst" <m.b.lankhorst(a)gmail.com> writes: > @@ -3059,12 +3060,14 @@ static INT HTTP_GetResponseHeaders(LPWININETHTTPREQW lpwhr) > * We should first receive 'HTTP/1.x nnn OK' where nnn is the status code. > */ > memset(buffer, 0, MAX_REPLY_LEN); > + memset(buffertoken, 0, MAX_REPLY_LEN); > if (!NETCON_getNextLine(&lpwhr->netConnection, bufferA, &buflen)) > goto lend; > MultiByteToWideChar( CP_ACP, 0, bufferA, buflen, buffer, MAX_REPLY_LEN ); > + MultiByteToWideChar( CP_ACP, 0, bufferA, buflen, buffertoken, MAX_REPLY_LEN ); That's ugly, it shouldn't be necessary to have two copies of the whole data just to parse some status codes. -- Alexandre Julliard julliard(a)winehq.org

1 0

Valgrind results. Update for glibc-2.6.
by Dan Kegel Feb. 25, 2008

Feb. 25, 2008

Today's valgrind results online at http://kegel.com/wine/valgrind/logs-2008-02-25-summary.txt http://kegel.com/wine/valgrind/logs-2008-02-25/ Nothing too special (thanks for the cleanup, James), but this is the first set of results after updating my test machine from feisty to gutsy, and I had to apply http://kegel.com/wine/valgrind/futimesat.patch to my copy of valgrind svn to handle the new glibc properly. Already reported to Valgrind; see http://sourceforge.net/mailarchive/forum.php?thread_name=a71bd89a0802240546…

3 3

Re: Add a slight delay before testing the ddraw and d3d9 visual tests
by Dmitry Timoshkov Feb. 25, 2008

Feb. 25, 2008

"Maarten Lankhorst" <m.b.lankhorst(a)gmail.com> wrote: > This makes the tests pass on my pc. Perhaps there is something less hacky way like GdiFlush() which is more appropriate for this. -- Dmitry.

2 3

Re: d3dx9_xx
by Francois Gouget Feb. 25, 2008

Feb. 25, 2008

On Fri, 22 Feb 2008, "Luis C. Busquets Pérez" wrote: > The number of functions reported in > http://www.winehq.org/site/winapi_stats > for the following dlls is > > d3dx9_24 308 [...] That's because d3dx9_36 has some forwards to functions that have not been declared in d3dx8, not even as a stub. It's allowed of course but winapi does not like it and thus ignores those. Here's the list: d3dx9_36.spec: forward is broken: D3DXFresnelTerm => d3dx8.D3DXFresnelTerm d3dx9_36.spec: forward is broken: D3DXGetImageInfoFromFileA => d3dx8.D3DXGetImageInfoFromFileA d3dx9_36.spec: forward is broken: D3DXGetImageInfoFromFileInMemory => d3dx8.D3DXGetImageInfoFromFileInMemory d3dx9_36.spec: forward is broken: D3DXGetImageInfoFromFileW => d3dx8.D3DXGetImageInfoFromFileW d3dx9_36.spec: forward is broken: D3DXGetImageInfoFromResourceA => d3dx8.D3DXGetImageInfoFromResourceA d3dx9_36.spec: forward is broken: D3DXGetImageInfoFromResourceW => d3dx8.D3DXGetImageInfoFromResourceW d3dx9_36.spec: forward is broken: D3DXIntersectSubset => d3dx8.D3DXIntersectSubset d3dx9_36.spec: forward is broken: D3DXIntersectTri => d3dx8.D3DXIntersectTri d3dx9_36.spec: forward is broken: D3DXQuaternionSquadSetup => d3dx8.D3DXQuaternionSquadSetup d3dx9_36.spec: forward is broken: D3DXSplitMesh => d3dx8.D3DXSplitMesh d3dx9_36.spec: forward is broken: D3DXTessellateNPatches => d3dx8.D3DXTessellateNPatches -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ You can have my guns when you pry them from my kids cold, dead hands.

1 0

How should we treat users on bugzilla?
by Dan Kegel Feb. 25, 2008

Feb. 25, 2008

I've been seeing a lot of bugs closed as INVALID when a poor user didn't file the bug perfectly. IMHO this leads to Wine having a user-hostile reputation. Case in point: http://bugs.winehq.org/show_bug.cgi?id=11625 What can we do to reduce this kind of behavior? Maybe we should have a "three warnings" sort of policy. If somebody continues to be rude to users after being asked twice not to, we should consider taking away their Bugzilla privileges for a while. Sure, that would hurt our productivity, but it might be better to do that than to alienate our users. What do other people think? - Dan

13 28

Re: WGL: wglDeleteContext should fail on garbage handles [attempt2]
by Alexandre Julliard Feb. 25, 2008

Feb. 25, 2008

"Roderick Colenbrander" <thunderbird2k(a)gmx.net> writes: > Subject: [PATCH] wglDeleteContext should fail on a garbage hglrc. When > we receive garbage ctx->hdc is invalid which leads to a > crash. Retrieve a default DC and use it to pass hglrc to the display > driver. Luckily the opengl code maintains a list of valid opengl > contexts, so it can detect whether the hglrc is valid or not. :) If we are going to go in this direction, then the context should be made opaque in gdi32, and all calls should use the default DC. The alternative is to make the context a proper GDI handle, and have GDI check it for validity first. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: winex11drv: Distinguish left and right keys for shift, ctrl and alt.
by Alexandre Julliard Feb. 25, 2008

Feb. 25, 2008

Vitaliy Margolen <wine-patches(a)kievinfo.com> writes: > --- a/dlls/winex11.drv/keyboard.c > +++ b/dlls/winex11.drv/keyboard.c > @@ -1022,9 +1022,9 @@ static const WORD nonchar_key_vkey[256] = > 0, 0, 0, 0, 0, 0, 0, 0, /* FFD0 */ > 0, 0, 0, 0, 0, 0, 0, 0, /* FFD8 */ > /* modifier keys */ > - 0, VK_SHIFT, VK_SHIFT, VK_CONTROL, /* FFE0 */ > - VK_CONTROL, VK_CAPITAL, 0, VK_MENU, > - VK_MENU, VK_MENU, VK_MENU, 0, 0, 0, 0, 0, /* FFE8 */ > + 0, VK_LSHIFT, VK_RSHIFT, VK_LCONTROL, /* FFE0 */ > + VK_RCONTROL, VK_CAPITAL, 0, VK_MENU, > + VK_MENU, VK_LMENU, VK_RMENU, 0, 0, 0, 0, 0, /* FFE8 */ > 0, 0, 0, 0, 0, 0, 0, 0, /* FFF0 */ > 0, 0, 0, 0, 0, 0, 0, VK_DELETE /* FFF8 */ This is going to change the vkeys everywhere, I don't think that's correct. Things like WM_KEYDOWN have to report VK_SHIFT, not VK_LSHIFT etc. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: winex11drv: Remove special treatment for caps-lock and num-lock keys. [try-2]
by Alexandre Julliard Feb. 25, 2008

Feb. 25, 2008

Vitaliy Margolen <wine-patches(a)kievinfo.com> writes: > - if (vkey) > - { > - switch (vkey & 0xff) > + if (!vkey) return; > + > + /* Adjust the NUMLOCK state if it has been changed outside wine */ > + if (!(key_state_table[VK_NUMLOCK] & 0x01) != !(event->state & NumLockMask)) > { > - case VK_NUMLOCK: > - KEYBOARD_GenerateMsg( VK_NUMLOCK, 0x45, event->type, event_time ); > - break; > - case VK_CAPITAL: > - TRACE("Caps Lock event. (type %d). State before : %#.2x\n",event->type,key_state_table[vkey]); > - KEYBOARD_GenerateMsg( VK_CAPITAL, 0x3A, event->type, event_time ); > - TRACE("State after : %#.2x\n",key_state_table[vkey]); > - break; > - default: > - /* Adjust the NUMLOCK state if it has been changed outside wine */ > - if (!(key_state_table[VK_NUMLOCK] & 0x01) != !(event->state & NumLockMask)) > - { > - TRACE("Adjusting NumLock state.\n"); > - KEYBOARD_GenerateMsg( VK_NUMLOCK, 0x45, KeyPress, event_time ); > - KEYBOARD_GenerateMsg( VK_NUMLOCK, 0x45, KeyRelease, event_time ); > - } > - /* Adjust the CAPSLOCK state if it has been changed outside wine */ > - if (!(key_state_table[VK_CAPITAL] & 0x01) != !(event->state & LockMask)) > - { > - TRACE("Adjusting Caps Lock state.\n"); > - KEYBOARD_GenerateMsg( VK_CAPITAL, 0x3A, KeyPress, event_time ); > - KEYBOARD_GenerateMsg( VK_CAPITAL, 0x3A, KeyRelease, event_time ); > - } > - /* Not Num nor Caps : end of intermediary states for both. */ > - NumState = FALSE; > - CapsState = FALSE; > + TRACE("Adjusting NumLock state (%#.2x).\n", key_state_table[VK_NUMLOCK]); > + X11DRV_send_keyboard_input( VK_NUMLOCK, 0x45, KEYEVENTF_EXTENDEDKEY, event_time, 0, 0 ); > + X11DRV_send_keyboard_input( VK_NUMLOCK, 0x45, KEYEVENTF_EXTENDEDKEY | KEYEVENTF_KEYUP, event_time, 0, 0 ); > + } > + /* Adjust the CAPSLOCK state if it has been changed outside wine */ > + if (!(key_state_table[VK_CAPITAL] & 0x01) != !(event->state & LockMask)) > + { > + TRACE("Adjusting Caps Lock state (%#.2x).\n", key_state_table[VK_CAPITAL]); > + X11DRV_send_keyboard_input( VK_CAPITAL, 0x3A, 0, event_time, 0, 0 ); > + X11DRV_send_keyboard_input( VK_CAPITAL, 0x3A, KEYEVENTF_KEYUP, event_time, 0, 0 ); > + } You still need to handle the case of caps lock or num lock keys differently, otherwise you'll generate duplicate messages. -- Alexandre Julliard julliard(a)winehq.org

1 0

SetPixelFormat broken in 0.9.54
by bill lam Feb. 25, 2008

Feb. 25, 2008

I run a opengl script that worked in 0.9.53 but crashed in 0.9.54 (and 0.9.55) I traced it failed at this line SetPixelFormat hdc;pixfmt;pfd with the following message in console: X Error of failed request: BadAccess (attempt to access private resource denied) Major opcode of failed request: 160 (Composite) Minor opcode of failed request: 1 () Serial number of failed request: 164 Current serial number in output stream: 182 Is this a known issue? ubuntu 7.10 amd64 regards,

2 1

GSOC (proposal)
by David Adam Feb. 25, 2008

Feb. 25, 2008

Here are my wishes for GSOC: D3DX dlls begin to be implemented. It seems that this could be a good GSOC to implement more feature of this dll, in particular the shader compiler. In the view of wined3d, I think about vertex blending, since there are tons of game using this feature. Stefan, is this doable in the scope of GSOC? Sound should be improved also, since there are games that still crash, in despite of M. Lankhorst's great job ( dmime, quartz,.... dlls problem). And the last but not the least :D: fix the infamous warping mouse bug. I know that it is mainly a problem with X server, but maybe a wine develooper could propose a GSOC to Xorg guys to fix this. David A.

3 2

AppDB stats
by Tony Lambregts Feb. 25, 2008

Feb. 25, 2008

Just a little note to all about the appdb stats. We now have 9000 application versions in the database. http://appdb.winehq.org/appdbStats.php Users: 23474 Comments: 21305 Application families: 5814 Versions: 9000 Application maintainers: 1462 Test reports: 18572 Screenshots: 6234 The Application maintainer stats is the most interesting in that we have 1462 individuals who now maintain applications. Most of these are regular users of wine and I for one really appreciate their effort in helping others. I would also like to recognize the individuals who have helped improve the AppDB code lately. Alexander Nicolaysen Sørnes Chris Morgan Jeff Zaroyko -- Tony Lambregts

1 0

re: GSoC project ideas
by Dan Kegel Feb. 25, 2008

Feb. 25, 2008

Kai wrote: > * Implementing a WinePluginApi so other programs can use > Windows DLLs inside of Linux apps That one's very tricky and systemy, a successful student would have to be a kernel and/or glibc maven, too. > * Valgrind and Wine integration (see: Wine_and_Valgrind) That one's sorted, or will be, so let's strike it. > * Run the Mauve Java test suite against Sun's Windows JRE, and file bugs / > write test cases in C / fix anything it finds Done. Mauve's a mess. We pass the part of it that makes sense. > * pick a Windows feature that is incomplete in Wine, and improve the feature > and its conformance tests. (Some good examples might be riched20 Believe it or not, there's still stuff to do there, I think. (Mostly COM-related?) - Dan -- Wine for Windows ISVs: http://kegel.com/wine/isv

1 0