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

Killing WineAPI project on SourceForge.
by Max TenEyck Woodbury Feb. 7, 2011

Feb. 7, 2011

No feedback and apparently no interest in the WineAPI project on SourceForge. Understandable since there is now a WineAPI on the regular Wine Wiki. Unless someone objects, I'm going to ask that it be shutdown on SourceForge. Max

5 6

Including winegcc-built shared library in Debian binary package
by Robert McDonald Feb. 7, 2011

Feb. 7, 2011

I've got a Debian binary package configured. A wrapper script is installed in /usr/bin that invokes wine on the winegcc-built shared library that implements the program. The shared library is installed in /usr/lib and then ldconfig is run during the installation process. The shared library build product seems to be a special case because it is built by winegcc and referenced on the wine command line when run. I've got questions about this even though the product works OK as packaged. Is there a more appropriate installation directory than /usr/lib for the shared library (although lintian complains if I don't put it in /usr/lib)? Does ldconfig need to be run after installing/uninstalling the shared library (other than to satisfy lintian complaints if ldconfig isn't run from postinst/postrm)? Is there any reason to bother with version numbering the shared library (other than using one version number for all library releases just to keep lintian happy by having the version number) since the library is prepared by winegcc and will only be loaded by wine? To look for answers, I've done the following: * searched the web * searched the WineHQ web site and read the FAQ there * scanned the subject lines of a couple of months of this mailing list * read http://wiki.winehq.org/Packaging * used https://wiki.ubuntu.com/PackagingGuide/Complete for some packaging instructions * read http://www.bytehold.com/index.php/projects/176-debian-binary-package-and-re… for general instructions in setting up a Debian binary package * read http://www.pathname.com/fhs/ for recommendations on locating files URLs to relevant resources would be welcome. -- Robert McDonald BitTorrent, Inc.

2 1

Re: iphlpapi: Changed return value of NotifyAddrChange() stub and added test cases.
by André Hentschel Feb. 7, 2011

Feb. 7, 2011

Hi, in todo_wine blocks only ok() calls should be made. So consider splitting them down to "todo_wine ok(..." for each. > -CancelIPChangeNotify > CreateProxyArpEntry > DeleteIPAddress > DeleteProxyArpEntry > EnableRouter > FlushIpNetTable > GetAdapterIndex > -NotifyAddrChange > -NotifyRouteChange > +NotifyRouteChange + CancelIPChangeNotify this change regarding CancelIPChangeNotify is also not needed. -- Best Regards, André Hentschel

2 1

Re: oleaut32: Fix copying an array of 0 elements
by Marvin Feb. 7, 2011

Feb. 7, 2011

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=8987 Your paranoid android. === WNT4WSSP6 (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W2KPROSP4 (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === WXPPROSP3 (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W2K3R2SESP2 (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === WVISTAADM (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W2K8SE (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W7PRO (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W7PROX64 (32 bit safearray) === safearray: unhandled exception c0000005 at 0040E5A5 === W7PROX64 (64 bit safearray) === Timeout

1 0

Patch to fix ntdll:exception tests on Mac OS
by Charles Davis Feb. 7, 2011

Feb. 7, 2011

Hi, This is a small patch I wrote to fix the exception tests on Mac OS X. Right now, if you look at the Mac results I submitted, you'll notice that there are two failures. The first failure happens because Mach doesn't allow the 'global' flag to be set when setting the debug registers. Changing the test not to set the global flag causes the test to pass. I get the feeling, though, that there is a good reason the global flag was set, so I'm reluctant to send that change in. The other failure happens because 32-bit Mac OS only protects stacks from execution. If you clear PROT_EXEC on any other data, you can still execute code from it. (64-bit Mac OS, on the other hand, always respects the PROT_EXEC flag.) I changed ntdll on 32-bit Mac OS to report DEP as permanently disabled, but I wonder if this was the right change to make. I just want to know: am I doing the right thing here? If not, what should I do instead to fix the tests? Chip

1 0

Re: user32: Don't try to move mouse to (-1,0).
by Vitaliy Margolen Feb. 6, 2011

Feb. 6, 2011

On 02/05/2011 07:10 AM, Jacek Caban wrote: > > This trigers screen edge action on KDE, which is 'Present Windows' in > default configuration. mouse_event() generates relative mouse movement events unless MOUSEEVENTF_ABSOLUTE is specified. If you don't want a pointer to get near the edge then you should use SetCursorPos() instead. Vitaliy.

2 1

re: Wine FAQ edits
by Dan Kegel Feb. 6, 2011

Feb. 6, 2011

Hi Albert, http://wiki.winehq.org/FAQ?action=diff&rev2=384&rev1=383 looks like it was a pretty big edit. Try smaller ones, and try to do just one kind of change per edit.

7 20

re: Pulling Patch
by Dan Kegel Feb. 6, 2011

Feb. 6, 2011

James McKenzie wrote: > Since my Mac is dying I have decided to return to the Windows world. > > Please pull any and all patches. I have envoked the right to copyright > and none of my code can or will be used in Wine. Sorry to see you go, but... why would you want to prohibit people from using your patches? (I only see two typo changes committed: http://source.winehq.org/git/wine.git/?a=search&h=HEAD&st=author&s=James+Mc… Presumably you don't mean those, since they were trivial comment changes.) I guess you mean patches like http://www.winehq.org/pipermail/wine-patches/2010-June/089537.html When you sent those to wine-patches, weren't you placing them under the LGPL? - Dan

4 5

RE: shell32/tests: Clean make new folder test (Valgrind)
by Michael Mc Donnell Feb. 5, 2011

Feb. 5, 2011

Hi! What is the reason this patch by Nikolay Sivov was not included in the latest version of wine? It fixes a clean-up bug on some Windows XP machines, and tidies up the code nicely. Wine-patches archive link: http://www.winehq.org/pipermail/wine-patches/2011-January/098342.html Regards, Michael Mc Donnell P.S. Thanks Nikolay for fixing my first win32 test.

1 0

Re: user32: Don't try to move mouse to (-1,0).
by Marvin Feb. 5, 2011

Feb. 5, 2011

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=8925 Your paranoid android. === W7PROX64 (64 bit msg) === Timeout

1 0

Pulling Patch
by James McKenzie Feb. 5, 2011

Feb. 5, 2011

Since my Mac is dying I have decided to return to the Windows world. Please pull any and all patches. I have envoked the right to copyright and none of my code can or will be used in Wine. Thank you. James McKenzie

2 1

Re: winspool.drv: make sure to use return values (LLVM/Clang)
by Marvin Feb. 5, 2011

Feb. 5, 2011

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=8921 Your paranoid android. === WINEBUILD (build) === Can't determine base name

1 0

Wine API documentation questions.
by Max TenEyck Woodbury Feb. 5, 2011

Feb. 5, 2011

I've been going through the Wine API documentation and there seem to be some things that I think should be changed: - There are references to Wine documents that do not include links to the pages. The links should be included. - Some 'implementation' sections claim that the API is not declared but there are declarations, just not in headers that 'c2man' is looking at. It should also look in any sub-directories (except for 'tests') of the '-I' directories it is told about for additional headers. (I have very little skill with Perl at present, so someone else might be able to add this much faster than I will be able to do it.) - There are quite a few APIs implemented but without standard documentation. All accessible APIs should have enough documentation so that there is a page for it, even if it is only a synopsis. - Mangled interface names only appear in their mangled form. The documentation should include the demangled name as well. Should the demangled names also appear in the indexes? - The 'synopsis' does not include needed '#include' directives. This may need to be fixed in each API description rather than trying to get 'c2man' to generate it. - The name of the file containing the implementation is included in each 'implementation' section and includes a link to the appropriate repository source entry. The name of the appropriate header is also included, but does not include a link. A link should be included. Max

1 0

Re: kernel32/process: Add stderr redirection for native linux programs started from windows program
by Austin English Feb. 5, 2011

Feb. 5, 2011

On Fri, Feb 4, 2011 at 18:22, Igor Egorov <atohom(a)gmail.com> wrote: > This is my first patch, not sure if I done all right. I didn't look at the actual change, but: > mode change 100644 => 100755 dlls/kernel32/process.c doesn't seem necessary to me. -- -Austin

2 1

Re: gdi32/tests: make sure to use return value (LLVM/Clang)
by Marvin Feb. 5, 2011

Feb. 5, 2011

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=8891 Your paranoid android. === WVISTAADM (32 bit font) === Failure running script in VM: The specified guest user must be logged in interactively to perform this operation

1 0

Tray icon not showing up if wine was run previously with DISPLAY unset?
by Dan Kegel Feb. 4, 2011

Feb. 4, 2011

So, I've got this shell script that does something like h=`wine winepath -w $HOME` ... wine autohotkey run-setup.ahk Works great. However, if the user has selected to run wine in a virtual desktop, running winepath will start up and then shut down the virtual desktop, which is visually annoying. Second try: unset DISPLAY when running winepath, e.g. h=`DISPLAY= wine winepath -w $HOME` ... wine autohotkey run-setup.ahk Works great; now winepath doesn't pop up the desktop. Except now autohotkey's tray icon doesn't show up at all, which makes it hard to debug the autohotkey script. (The setup that autohotkey runs can access X just fine, so the problem seems confined to the tray icon.) Is the lack of tray icon a wine bug, or a user error? Third try: wait for wineserver to exit before starting autohotkey h=`DISPLAY= wine winepath -w $HOME` ... wineserver -w wine autohotkey run-setup.ahk but that's slower, and will hang if the user is running a wine app. I guess I could do "sleep 5" but that's an awful kludge. Probably I'll go back to letting the ugly virtual desktop pop up. So, what's the proper way to run one app under wine *without* X while letting other apps run wine *with* X? - Dan

2 1

Re: loader: Build the preloader for ARM
by Alexandre Julliard Feb. 4, 2011

Feb. 4, 2011

André Hentschel <nerv(a)dawncrow.de> writes: > +unsigned int wld_modulo( const unsigned int a, const Elf32_Word b ) > +{ > +#ifdef __ARM_EABI__ > + unsigned int i; > + if(b > a) return a; > + for(i = 0; i * b < a + b; i++) > + if(i * b > a - b) return (a - i * b); > + return 0; That's very inefficient, there are better ways... -- Alexandre Julliard julliard(a)winehq.org

4 5

the patches around ok()
by Joris Huizer Feb. 4, 2011

Feb. 4, 2011

Hello Henri Verbeet, and André Hentschel I wasn't (actively) on the mailing list, that's why I couldn't reply directly to your messages. I will try and make sure I have better titles for such patches in future! Regards, Joris

2 1

Re: kernel32/tests/thread.c: Don't call in ok()
by Henri Verbeet Feb. 4, 2011

Feb. 4, 2011

On 4 February 2011 17:15, Joris Huizer <joris_huizer(a)yahoo.com> wrote: > Subject: kernel32/tests/thread.c: Don't call in ok() Why not?

2 1

Try to Implement my first Stub function - AbortPrinter().
by Loïc Maury Feb. 4, 2011

Feb. 4, 2011

Hello, I try to implement my first stub function - *AbortPrinter()*. But before to make a patch, I need your advice. For what I understood, AbortPrinter(), remove the document spool file for a printer, created by StartDocPrinter(), who indicate that a document was spooled. I saw that the API printer functions are in dlls/winspool.drv/info.c I understood that there are 4 structures used, *opened_printer_t* for manage a printer, *jobqueue_t* for a list of job for a printer, *started_doc_t* who manage a document, and *job_t* for manage a job. I am on Mac Os 10.6, with CUPS. Here is the code, what do you think ? BOOL WINAPI AbortPrinter(HANDLE hPrinter) { PRINTER_INFO_5W *pi5 = NULL; BOOL ret = FALSE; opened_printer_t *printer; struct list *cursor, *cursor2; job_t *job = 0; DWORD jobDocId; DWORD needed; LPWSTR portname; #if defined(__APPLE__) EnterCriticalSection(&printer_handles_cs); printer = get_opened_printer(hPrinter); if(!printer) { ERR("The handle for the printer is invalid.\n"); SetLastError(ERROR_INVALID_HANDLE); goto end; } TRACE("(%s, %s, %p, %d, %d)\n",debugstr_w(printer->name) ,debugstr_w(printer->printername) ,printer->backend_printer ,printer->queue->ref ,list_count(&printer->queue->jobs)); /* No jobs to manage. */ if(list_count(&printer->queue->jobs) == 0) { ERR("No job in the list.\n"); SetLastError(ERROR_SPL_NO_STARTDOC); goto end; } if(printer->doc) { TRACE("Document inside for job id : %d\n", printer->doc->job_id); jobDocId = printer->doc->job_id; } else { ERR("No document.\n"); SetLastError(ERROR_SPL_NO_STARTDOC); goto end; } /* For each jobs, get the job document in the double linked list */ LIST_FOR_EACH_SAFE(cursor, cursor2, &printer->queue->jobs) { /* Take a job. */ job = LIST_ENTRY(cursor, job_t, entry); TRACE("(job id : %d, filename : %s, portname : %s, document title : %s, printer name %s)\n",job->job_id ,debugstr_w(job->filename) ,debugstr_w(job->portname) ,debugstr_w(job->document_title) ,debugstr_w(job->printer_name)); if(jobDocId == job->job_id) { TRACE("(hf : %p, job id : %d)\n",printer->doc->hf ,printer->doc->job_id); /* Get portname. */ if (!job->portname) { GetPrinterW(hPrinter, 5, NULL, 0, &needed); pi5 = HeapAlloc(GetProcessHeap(), 0, needed); GetPrinterW(hPrinter, 5, (LPBYTE)pi5, needed, &needed); portname = pi5->pPortName; } TRACE("portname : %s\n", debugstr_w(portname)); /* Cups Port */ if(!strncmpW(portname, CUPS_Port, strlenW(CUPS_Port))) { TRACE("Remove job from the list.\n"); list_remove(cursor); CloseHandle(printer->doc->hf); DeleteFileW(job->filename); HeapFree(GetProcessHeap(), 0, job->document_title); HeapFree(GetProcessHeap(), 0, job->printer_name); HeapFree(GetProcessHeap(), 0, job->portname); HeapFree(GetProcessHeap(), 0, job->filename); HeapFree(GetProcessHeap(), 0, job->devmode); HeapFree(GetProcessHeap(), 0, job); job = 0; ret = TRUE; if(pi5) HeapFree(GetProcessHeap(), 0, pi5); } else FIXME("only CUPS for now.\n"); } } end: LeaveCriticalSection(&printer_handles_cs); TRACE("return %d\n", ret); return ret; #else TRACE("Only CUPS for Mac Os is managed for now.\n"); return FALSE; #endif } Merci Loïc

4 4

Re: msvcr90: Fixed a typo
by Nikolay Sivov Feb. 4, 2011

Feb. 4, 2011

On 2/4/2011 18:33, André Hentschel wrote: > --- > dlls/msvcr90/msvcr90.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/msvcr90/msvcr90.c b/dlls/msvcr90/msvcr90.c > index 2c811cb..d2a61dd 100644 > --- a/dlls/msvcr90/msvcr90.c > +++ b/dlls/msvcr90/msvcr90.c > @@ -315,7 +315,7 @@ const char * __thiscall MSVCRT_type_info_name_internal_method(type_info * _this, > if (!_this->name) > { > /* Create and set the demangled name */ > - /* Nota: mangled name in type_info struct always start with a '.', while > + /* Note: mangled name in type_info struct always start with a '.', while > * it isn't valid for mangled name. > * Is this '.' really part of the mangled name, or has it some other meaning ? > */ Maybe 'starts with' ?

2 1

Re: user32: Add a test for reparenting a WS_POPUP window to a WS_CHILD parent.
by Alexandre Julliard Feb. 4, 2011

Feb. 4, 2011

Dmitry Timoshkov <dmitry(a)codeweavers.com> writes: > + SetLastError(0xdeadbeef); > +todo_wine > + ok(SetForegroundWindow(popup), "SetForegroundWindow() error %d\n", GetLastError()); It would be interesting to test what the foreground window is actually set to. > + SetLastError(0xdeadbeef); > + ok(DestroyWindow(parent), "DestroyWindow() error %d\n", GetLastError()); Please don't add invalid uses of GetLastError again, this was cleaned up recently. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: user32: Add a test for reparenting a WS_POPUP window to a WS_CHILD parent.
by Joris Huizer Feb. 4, 2011

Feb. 4, 2011

Hello Dmitry Timoshkov, I noticed a small issue in your patch: > > - ret = DestroyWindow(parent); > - ok( ret, "DestroyWindow() error %d\n", > GetLastError()); > + ok(DestroyWindow(parent), "DestroyWindow() > error %d\n", GetLastError()); Please don't merge the call with the ok() statement; the GetLastError() call might be executed before the function to be tested, thus giving the old value. (There has been some cleaning up for this issue recently) The same goes for a few tests you added: > + SetLastError(0xdeadbeef); > + ok(SetForegroundWindow(popup), > "SetForegroundWindow() error %d\n", GetLastError()); > + SetLastError(0xdeadbeef); > +todo_wine > + ok(SetForegroundWindow(popup), > "SetForegroundWindow() error %d\n", GetLastError()); > + > + SetLastError(0xdeadbeef); > + ok(DestroyWindow(parent), "DestroyWindow() > error %d\n", GetLastError()); > + HTH, Joris

2 1

RE: Correction to crash inside RtlCaptureStackBackTrace() + test case
by Janne Hakonen Feb. 4, 2011

Feb. 4, 2011

> > It doesn't need a try-except block, just better wrap-around handling. > Ah, you mean to enhance the if-check? Well..., to me it seems that there is a problem with the 2nd check: "(void *)(frame + 1) >= NtCurrentTeb()->Tib.StackBase" If frame is 0xffffffff or close to it (e.g. 0xfffffffc), adding 1 to its address makes it overflow and the left side becomes 0x3, or 0x0 in 0xfffffffc's case. And that would make the check fail. I would propose that this would be corrected with an additional overflow check. There are two 4 byte values that needs to be read: next frame address and return address. So, it would be checked that address of last byte of return address (frame + 7 bytes) is not smaller than current frame address, right? The if-check would then be as follows: if (((void *)frame < NtCurrentTeb()->Tib.StackLimit) || ((void *)(frame + 1) >= NtCurrentTeb()->Tib.StackBase) || ((ULONG_PTR)frame & 3) || ((ULONG)((char *)frame + 7) < (ULONG)frame)) ... I'm not quite sure if casting pointers to ULONG for comparison is wise move, but can't think anything else for that. Do you think there is any problems with this approach? Other alternative could be to just simply check if the frame pointer equals to 0xfffffffc, that, combined with "(ULONG_PTR)frame & 3" should handle all cases near 0xffffffff. Best Regards, Janne

2 3

Re: wined3d/arb_program_shader: Small optimization in state_texfactor_arbfp().
by Stefan Dösinger Feb. 4, 2011

Feb. 4, 2011

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am 03.02.2011 um 23:01 schrieb Adam Martinson: > > Oddly enough this gives us an average 1.8% boost in the 3DMark06 batch size tests. Impressive. One more thing you could investigate(but that doesn't affect this patch): Try to figure out why this code is called in the first place. The only situation when this should happen is if the app is using fixed function fragment processing. I wouldn't expect that 3Dmark06 does that. -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.17 (Darwin) iQIcBAEBAgAGBQJNS8aYAAoJEN0/YqbEcdMwI1AQAJder+8LZILzdT+8tA2YiwZ+ ZEVOi4S2voFASHUVJbYuWIXkzzVu6fQlZXw/KCyvdQaslpTWQFFc/cOMXcCVjVs5 puJJDydjCyI8FZF7ec8dCIjCHqBH4S91kxDjRmWGifdp1Qd6h/f7o1xyhOHG62xG XZbY1xx3DhQ7lRxRtsdqqq22MM5cOqR4mCen2YPGsd04rlwsxW6zuxZ32oBof0rO ggzBW1F2WdDHeT6rcDK5Iv7R4zdSCtl6IXfsMaT1frTcd8Mo/9jLj7GaLMnQ/UtG JkyQbCP59AOtEKU8nwrISn71YMRkCBHOttaIAVhr6Nq02eNmcxWomSbIsb293LzP fQHFDPJ2aO161AimO/7Nk6DbGniUtlgRGhPhrK+YJvrAhftrtZ2gFUaN3QLJgcKA HHhkU00NsBXOJvz8YuGFDRVmpPpCLcGM7gvMIyjYy3x3W1FWO4giNxnj6CxqHmLs ohkAbsLEnJohhpTyVEaQKTG0Zq49s5WcvkjQbmncqKdP2D9NHmi50FI8MbJJxXAu mMW6sA6oExn31Fnm1/pQOWCZk8hidK1hRN7f25nSRa58q7bSvNT/Gw3kk6S8T5CU yL3r5UIb64ss4QmtXY8SPIR9IDNvmnYUX4/Av/iKpZrSRHCHzpYX7Eecp2jdV/ta jgKpR4VaDyemyij093oW =LXas -----END PGP SIGNATURE-----

1 1