March 2013 - Wine-Devel - list.winehq.org

Re: [3/3] Make mac driver the default on OS X
by Ken Thomases March 6, 2013

March 6, 2013

On Mar 5, 2013, at 6:32 PM, Josh DuBois wrote: > diff --git a/dlls/winemac.drv/macdrv_main.c b/dlls/winemac.drv/macdrv_main.c > index cd1cc0c..1f16694 100644 > --- a/dlls/winemac.drv/macdrv_main.c > +++ b/dlls/winemac.drv/macdrv_main.c > @@ -85,7 +85,7 @@ static BOOL process_attach(void) > macdrv_err_on = ERR_ON(macdrv); > if (macdrv_start_cocoa_app(GetTickCount64())) > { > - ERR("Failed to start Cocoa app main loop\n"); > + TRACE("Failed to start Cocoa app main loop\n"); > return FALSE; > } > I would think that this should still be at least a WARN, although I'm not sure why it's being demoted from ERR in the first place. (In fact, I should probably have added a WARN in my patch for refusing to run in non-GUI sessions.) -Ken

1 0

Re: msxml3: Implement IXMLParser Get/Set Factory
by Nikolay Sivov March 5, 2013

March 5, 2013

On 3/6/2013 09:11, Alistair Leslie-Hughes wrote: > Hi, > > > Changelog: > msxml3: Implement IXMLParser Get/Set Factory You also need to release factory reference on parser release it seems. > > > Best Regards > Alistair Leslie-Hughes > >

1 0

Re: comctl32/imagelist: use proper color format for merged image lists (try 2)
by Alexandre Julliard March 5, 2013

March 5, 2013

Daniel Jelinski <djelinski1(a)gmail.com> writes: > +static void test_merge_colors(void) > +{ > + HIMAGELIST himl[5], hmerge; > + int sizes[] = { ILC_COLOR4, ILC_COLOR8, ILC_COLOR16, ILC_COLOR24, ILC_COLOR32 }; You should also test COLORDDB, and probably the mask behavior too. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: mmdevapi: Prevent 64 bit overflow within a few days of audio device use. (try 2)
by Michael Stefaniuc March 5, 2013

March 5, 2013

Hello Joerg, On 03/01/2013 10:22 AM, Joerg-Cyril.Hoehle(a)t-systems.com wrote: > The idea to replace X * numerator / denominator > by X / den * mul + remainder from euclidian division > came from > http://blog.airsource.co.uk/index.php/2010/03/15/quelle-heure-est-il/ > > M. Stefaniuc suggested an inline function. The MulDiv64 inline has the benefit > of ensuring unsigned arithmetic, matching GetPosition UINT64 output. My idea was to have the whole if else in an inline function. But I see that even the whole functionality can move to a helper. Something like this: static inline ULONGLONG get_qpctime(void) { LARGE_INTEGER stamp, freq; QueryPerformanceCounter(&stamp); QueryPerformanceFrequency(&freq); if(freq.QuadPart == 10000000) return stamp.QuadPart; else return stamp.QuadPart / freq.QuadPart * 10000000 + stamp.QuadPart % freq.QuadPart * 10000000 / freq.QuadPart; } bye michael

2 2

Re: [PATCH 2/2] winmm: MCI_BREAK_HWND handling now matches Wine's 1999 MCI_DefYieldProc.
by Alexandre Julliard March 5, 2013

March 5, 2013

<Joerg-Cyril.Hoehle(a)t-systems.com> writes: > @@ -2063,9 +2063,10 @@ static DWORD MCI_Break(UINT wDevID, DWORD dwFlags, LPMCI_BREAK_PARMS lpParms) > switch (dwFlags & (MCI_BREAK_KEY | MCI_BREAK_OFF)) { > case MCI_BREAK_KEY: > wmd->lpfnYieldProc = MCI_DefYieldProc; > - wmd->dwYieldData = lpParms->nVirtKey; > - if (dwFlags & MCI_BREAK_HWND) > - FIXME("(%x) vkey %02X on window %p\n", dwFlags, lpParms->nVirtKey, lpParms->hwndBreak); > + /* Our MCI_DefYieldProc from 1999-04-11 expects this DWORD splitting, > + * yet tests on w2k-w7 prove wmd->dwYieldData = lpParms->nVirtKey; */ It doesn't make sense to add workarounds for Wine code, it should be fixed instead. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: server/advapi32: Report administrator ownership by default for registry objects (try 5).
by Alexandre Julliard March 5, 2013

March 5, 2013

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > +static struct security_descriptor *key_get_sd( struct object *obj ) > +{ > + struct security_descriptor *sd = default_get_sd( obj ); > + const SID *admins_sid = security_builtin_admins_sid; > + int admin_len; > + char *ptr; > + > + if (sd) return sd; > + admin_len = FIELD_OFFSET(SID, SubAuthority[admins_sid->SubAuthorityCount]); > + sd = mem_alloc( sizeof(struct security_descriptor) + 2*admin_len ); > + if (!sd) return sd; > + > + sd->control = 0; > + sd->owner_len = admin_len; > + sd->group_len = admin_len; > + sd->sacl_len = 0; > + sd->dacl_len = 0; > + > + ptr = (char *)(sd + 1); > + memcpy( ptr, security_builtin_admins_sid, sd->owner_len ); > + ptr += sd->owner_len; > + memcpy( ptr, security_builtin_admins_sid, sd->group_len ); > + ptr += sd->group_len; > + > + free( obj->sd ); > + obj->sd = sd; > + return sd; > +} It shouldn't be necessary to allocate one for every key. -- Alexandre Julliard julliard(a)winehq.org

1 0

Making wine default to the Mac Driver on OS X
by Josh DuBois March 5, 2013

March 5, 2013

With Friday's release, I am interested in making the Mac Driver wine's default on OS X. My own immediate motivation is largely so we at CodeWeavers can start testing it as a default internally (which obviously we can do on our own, regardless, if needed), but I figured I'd ask the wider community to see whether it's time in winehq also. My question is how it should be configured from a user perspective. Right now one can set a registry key in HKCU/Software/Wine/Drivers/Graphics with a comma-separated list specifying the driver load order, and the first one successfully loaded will be used. Absent the key, there is a hardcoded load order in user32/driver.c. Right now that is just "x11." Discussing with Alexandre the other day I believe his thought is to make the hardcoded default "x11,mac" in all cases (including Linux). I believe (and I don't mean to put words in his mouth) that his thought is that a user should never have to edit the registry to switch between drivers. The thought would be that in order to switch between drivers, a user must instead create an environment in which all drivers which are ahead of the desired driver in the hardcoded load order will fail to load. I think that in the specific case of "x11,mac" on an OS X box, the thought was to unset DISPLAY if you want to run with the Mac Driver on, forcing X11 to fail and therefore getting the fallback. I just now realized, though, that OS X 10.8 ships with stub X11 libraries which behave badly. If you try to load X11 on a brand-new OS X 10.8 machine, the user sees a dialog box with directions on downloading XQuartz, and the process which tried to load X11 is killed. I hadn't thought of that earlier, but it seems to weigh against using a fallback mechanism where x11 is attempted first in all cases. It means that to get good behavior from the fall-through, a Mac user would be required to download and install real X libraries, then unset DISPLAY. It seems broadly incorrect to me to attempt to load X11 first everywhere and force the user to make sure loading that fails if they want a different driver. I think that if the Mac Driver is to be the default on a Mac, rather than assume the environment will be such that other drivers will fail, it should really just be the first thing wine tries to load. The case of OS X's bad X11 stub libraries is one where it seems fairly onerous for a user to ensure X11 won't load. Wayland isn't here now, but it seems like when it comes it may bring similar issues. I propose an environment variable, WINEDRIVERLOADORDER. I would think we would first use the registry value, then the value of WINEDRIVERLOADORDER, then the hardcoded fallback. I would also propose making the hardcoded fallback differ on OS X and linux, so that a user who does nothing on OS X can have the Mac Driver, but someone who wants X11 can set WINEDRIVERLOADORDER and switch to that without editing the registry. Is that an acceptable approach? Of course, Ken may abhor the idea of having the Mac Driver the default in winehq at this point, anyway. ;)

5 7

wanted: testbot w9x machines
by Joerg-Cyril.Hoehle＠t-systems.com March 4, 2013

March 4, 2013

Hi, Francois Gouget wrote in another thread: >Not so long ago that machine was also running the tests on 5 additional >NT4 configurations, Windows 95, Windows 98 and Windows Millenium. It no >longer does, not because of lack of 'resources' or 'intention', but >because the Wine community is not interested in these. Huh? What? Please, PLEASE give me a (new)testbot with some w9x machines such that I can investigate the winmm&mci behavior that the apps written in that era encountered. It really slows down development on my side *not* to have an easily checked w9x system (nor a system for 16 bit tests, BTW). All that I can remember when that issue was raised last time was: 1. It's good not to test incoming patches against w9x behavior, because the time spent adding broken() for those is better invested adding code and more tests for w2k-w7 machines. Agreed. 1' test.winehq.org need no more display w9x test results. I don't care much. 2. Some technical aspects, such as not being able to find Greg's old w95 VM image, or producing a VM image with some soundcard not hanging randomly. But I can't remember an argument saying like "nobody was interested in those w95/w98/w98SE/wME machines, so we dumped them". Currently, (new)testbot doesn't even offer an option to run tests on w9x machines. I'm not asking for that option to be on by default, I'm just begging for it to be present! Thank you, Jörg Höhle

2 1

Re: gdiplus/tests: Skip the tests if unable to load the TIFF image.
by Dmitry Timoshkov March 4, 2013

March 4, 2013

Francois Gouget <fgouget(a)codeweavers.com> wrote: > On Windows XP most versions of gdiplus (i.e. older than 5.2) do not support TIFF. I'd suggest to upgrade the VM instead. Ideally each VM should represent a Windows version with newest service pack installed it's supposed to run the tests on, otherwise that VM is useless. IMHO there is no point in changing the tests to workaround a broken VM state. -- Dmitry.

3 7

Re: [1/3] Revert "gdiplus: Add support for StringFormatFlagsNoClip."
by Vincent Povirk March 4, 2013

March 4, 2013

Yay, thanks for fixing this. +1 :D

1 0

Use accented letters in their names.
by Tae Wong March 4, 2013

March 4, 2013

You will want to use accented letters in their authors names. Caolan McNamara Ferenc Wagner Marko Nikolic Every once and a while the GNOME Live website is down with an error on Netscape called “The operation timed out when attempting to contact .” - This paragraph should have a website name. You want to be unbanned in Amaterasu Translations and the DOS ain’t dead forums.

2 2

Re: comctl32/imagelist: use DDB instead of 16-color bitmaps
by Alexandre Julliard March 4, 2013

March 4, 2013

Daniel Jelinski <djelinski1(a)gmail.com> writes: > Subject: [PATCH 1/2] comctl32/imagelist: use DDB instead of 16-color bitmaps > > I don't know if DDB is a good option here, but 32bit color resulted in > invisible images, and anything is better than the current 16 colors Some test cases would be a good idea. -- Alexandre Julliard julliard(a)winehq.org

1 0

IOCTL_DVD_READ_STRUCTURE (DvdPhysicalDescriptor) test failure
by Detlef Riekenberg March 4, 2013

March 4, 2013

2 1

IOCTL_DVD_READ_STRUCTURE (DvdPhysicalDescriptor) test failure [text only]
by Detlef Riekenberg March 3, 2013

March 3, 2013

Hi Alexandre. I have a test failure on my win7 Machine, while a valid cd was in the dvd drive. volume.c:785: Test failed: IOCTL_DVD_READ_STRUCTURE (DvdPhysicalDescriptor) failed, last error = 1785 Example result: http://test.winehq.org/data/209b58c64e6181c9bf448cb4f3130d3ba705c5d2/win7_d… (The same failure is present, when running the 64bit test) Other machines are also affected. Example: http://test.winehq.org/data/4a685ebea343efdcb433f8d710b0c9f902e471fa/win7_m… You wrote the tests, which was comitted on 12. July 2012: http://source.winehq.org/git/wine.git/commitdiff/a3b5dd2dff0d04fe317f00282c… What media did you use, to test your code? Data-DVD, Video-DVD, Empty DVD, Data-CD, Video-CD, Audio-CD, Empty CD, No Disc in Drive What results did you get on Windows and Unix? On Linux, mounting the example CD works perfect, but your test fails: volume.c:785: Test failed: IOCTL_DVD_READ_STRUCTURE (DvdPhysicalDescriptor) failed, last error = 87 Your test only read the DVD descriptor. To fix your test, I suggest, that you add a trace to dump the *_READ_STRUCTURE IoControl result and add similar tests for the other media formats. I also tested with a different Data-CD and got the same failure on linux (last error = 87). You can check the image yourself: http://www.memtest86.com/downloads/memtest86-4.1.0-iso.zip (Burned on a CD) Your broken test was not detected by winetest, but that is a different issue http://bugs.winehq.org/show_bug.cgi?id=31780 sorry for the prev. html mail -- By by ... Detlef

1 0

Re: [PATCH] Add support for -Ttext-segment linker flag to set image base of a dll.
by Alexandre Julliard March 2, 2013

March 2, 2013

Tijl Coosemans <tijl(a)coosemans.org> writes: > --- > configure.ac | 2 ++ > tools/winegcc/winegcc.c | 8 ++++++++ > 2 files changed, 10 insertions(+) This would have to be done based on the target platform, not on a configure check. -- Alexandre Julliard julliard(a)winehq.org

2 1

FYI ubuntu PPA 64-bit did not update wine1.5-386
by Susan Cragin March 2, 2013

March 2, 2013

just letting you know. assume it will be available later.

1 0

Re: gdiplus: GdipDrawString should handle alignment in both directions in a similar way. Take 2.
by Vincent Povirk March 1, 2013

March 1, 2013

Sorry, I'm still not sure this is going to work. It seems to me that different lines will have to be offset by different amounts, and it's not clear to me how that's going to happen without the alignment logic in gdip_format_string. I'm also wondering if a change to the alignment logic in gdip_format_string is going to break MeasureCharacterRanges.

1 0

Re: [PATCH 3/3] msvcrt: Added stdin close/open tests
by Marvin March 1, 2013

March 1, 2013

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

1 0

Re: d3dx9 [patch 1/2]: Implement D3DXSHEvalConeLight
by Marvin March 1, 2013

March 1, 2013

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

3 2

Re: ieframe: Fixed tests on IE10.
by Marvin March 1, 2013

March 1, 2013

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=24562 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === Timeout === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2613: Test failed: expected GetOverridesKeyPath webbrowser.c:2618: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2619: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3052: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402F21

1 0