October 2012 - Wine-Devel - list.winehq.org

Re: jscript: Don't release possible NULL pointer (coverity)
by Jacek Caban Oct. 18, 2012

Oct. 18, 2012

Hi André, On 10/18/12 00:14, André Hentschel wrote: > CID 735258 > > --- > dlls/jscript/regexp.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/dlls/jscript/regexp.c b/dlls/jscript/regexp.c > index bdff445..71474c7 100644 > --- a/dlls/jscript/regexp.c > +++ b/dlls/jscript/regexp.c > @@ -3378,7 +3378,9 @@ static HRESULT do_regexp_match_next(script_ctx_t *ctx, RegExpInstance *regexp, D > last_match = jsstr_alloc_len(str, len); > if(!last_match) > return E_OUTOFMEMORY; > - jsstr_release(ctx->last_match); > + > + if(ctx->last_match) > + jsstr_release(ctx->last_match); > ctx->last_match = last_match; ctx->last_match should always be a valid string. I guess coverity is confused by earlier compare to NULL, which is a leftover after string representation change. We should use jsstr_t here to avoid allocation anyway, I will send a patch. Thanks, Jacek

1 0

Re: (resend) programs/winemenubuilder: scale 64x64 classic icons to 48x48 for Icns format
by Ken Thomases Oct. 18, 2012

Oct. 18, 2012

On Oct 16, 2012, at 2:15 PM, Aric Stewart wrote: > @@ -1052,6 +1079,7 @@ static inline int size_to_slot(int size) > case 16: return 0; > case 32: return 1; > case 48: return 2; > + case 64: return 2; /* Classic Mode */ > case 128: return 3; > case 256: return 4; > case 512: return 5; If an ICO has both a 48x48 and a 64x64 entry (improbable, I know), then the 48x48 should be preferred. So, you need additional logic to pick the best index for a slot in platform_write_icon(). Track the best-so-far's width and prefer the smaller. Beyond that, perhaps the scaling should be put into the ICNS encoder rather than convert_to_native_icon(). That is, if it's handed a size that doesn't fit properly into one of the predefined slots, it would scale down to the next lower size. Although it might be better to synthesize a 32x32 rather than a 48x48 from a 64x64. *shrug* -Ken

5 6

Re: advapi32: Only require a domain in the username for CRED_PERSIST_ENTERPRISE credentials.
by Marvin Oct. 18, 2012

Oct. 18, 2012

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=22306 Your paranoid android. === WXPPROSP3 (32 bit cred) === cred.c:128: Test failed: CredWriteA failed with error 2202 cred.c:131: Test failed: CredDeleteA failed with error 1168 cred.c:135: Test failed: CredWriteA failed with error 1312 cred.c:138: Test failed: CredDeleteA failed with error 1168 cred.c:149: Test failed: CredWriteA failed with error 2202 cred.c:152: Test failed: CredDeleteA failed with error 1168 === W2K3R2SESP2 (32 bit cred) === cred.c:128: Test failed: CredWriteA failed with error 2202 cred.c:131: Test failed: CredDeleteA failed with error 1168 cred.c:135: Test failed: CredWriteA failed with error 1312 cred.c:138: Test failed: CredDeleteA failed with error 1168 cred.c:149: Test failed: CredWriteA failed with error 2202 cred.c:152: Test failed: CredDeleteA failed with error 1168

1 0

Re: msvcp: free facet and _Locimp in Locimp dtor
by Piotr Caban Oct. 18, 2012

Oct. 18, 2012

On 10/17/12 02:35, Daniel Lehman wrote: @@ -7241,10 +7241,11 @@ void __thiscall locale__Locimp_dtor(locale__Locimp *this) MSVCP_size_t i; for(i=0; i<this->facet_cnt; i++) if(this->facetvec[i] && locale_facet__Decref(this->facetvec[i])) - call_locale_facet_vector_dtor(this->facetvec[i], 0); + call_locale_facet_vector_dtor(this->facetvec[i], 1); MSVCRT_operator_delete(this->facetvec); MSVCP_basic_string_char_dtor(&this->name); + MSVCRT_operator_delete(this); } } Memory allocated for locale__Locimp object should be freed in locale class (locale_dtor function). You can't delete it here. Cheers, Piotr

2 4

Re: gdiplus: fix font test when liberation fonts are installed
by Dmitry Timoshkov Oct. 18, 2012

Oct. 18, 2012

Austin English <austinenglish(a)gmail.com> wrote: > + ok(!lstrcmp(lf.lfFaceName, "Arial") || > + !lstrcmp(lf.lfFaceName, "Liberation Sans"), "wrong face name %s\n", lf.lfFaceName); The tests are supposed to reflect Windows behaviour, what Windows version does return "Liberation Sans" in this test? -- Dmitry.

2 2

re: [PATCH 4/6] [cmd] Add for /f parsingkeyword parsing, add support for skip=
by Dan Kegel Oct. 17, 2012

Oct. 17, 2012

Jason wrote: >>It's clearly time to start splitting WCMD_for into several helper functions. > > I completely agree, and it was definitely on my radar to do... its really > ugly code in there (I take responsibility for a lot of that!) as it has > been a slow incremental increase in what we actually supported... I was > aiming to get a set of working /f tests before I did such a reshuffle as > that is the last of the functionality that for was missing (ie I could > ensure I keep it all working), but will prioritize it if you want it before > that. Seems like it makes sense to add as many tests as possible before refactoring, but if they're messy to submit without also fixing the problems they find, go ahead and keep them local for now... just don't forget to submit them asap. I must say, it's awesome how you're tearing into the cmd problems! - Dan

1 0

Re: shell32/tests: Test ShellExecute()'s URL handling.
by Marvin Oct. 17, 2012

Oct. 17, 2012

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=22302 Your paranoid android. === WNT4WSSP6 (32 bit shlexec) === shlexec.c:1800: Test failed: ShellExecute(file="notaproto://foo") succeeded: rc=2 shlexec.c:1803: Test failed: ShellExecute(file="fakeproto://foo/bar") expected SE_ERR_ACCESSDENIED got rc=2 shlexec.c:152: Test failed: ShellExecute(file="shlproto://foo/bar") Got different return value with empty string: 42 2 shlexec.c:152: Test failed: ShellExecute(file="shlproto://foo/bar.lnk") Got different return value with empty string: 42 2 shlexec.c:137: Test failed: ShellExecute(file="file://shlproto://foo/bar") WaitForSingleObject returned 258 shlexec.c:152: Test failed: ShellExecute(file="file://shlproto://foo/bar") Got different return value with empty string: 42 2 shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=42 === W2KPROSP4 (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=5 === WVISTAADM (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === W2K8SE (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === W7PRO (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === W7PROX64 (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === TEST64_W7SP1 (32 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === W7PROX64 (64 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2 === TEST64_W7SP1 (64 bit shlexec) === shlexec.c:1843: Test failed: ShellExecute(file="file://shlproto://foo/bar") succeeded: rc=2

1 0

Re: [PATCH] winmm: Load winealsa if winepulse is found
by Austin English Oct. 17, 2012

Oct. 17, 2012

On Tue, Oct 16, 2012 at 5:10 AM, Maarten Lankhorst <m.b.lankhorst(a)gmail.com> wrote: > From: Maarten Lankhorst <maarten.lankhorst(a)canonical.com> > > Fixes midi on winepulse > --- > dlls/winmm/lolvldrv.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/dlls/winmm/lolvldrv.c b/dlls/winmm/lolvldrv.c > index f387323..3b1be27 100644 > --- a/dlls/winmm/lolvldrv.c > +++ b/dlls/winmm/lolvldrv.c > @@ -543,7 +543,10 @@ static void MMDRV_Init(void) > drvA = HeapAlloc(GetProcessHeap(), 0, size); > WideCharToMultiByte(CP_ACP, 0, pv.u.pwszVal, -1, drvA, size, NULL, NULL); > > - MMDRV_Install(drvA, drvA, FALSE); > + if (!strcasecmp(drvA, "winepulse.drv")) > + MMDRV_Install("winealsa.drv", "winealsa.drv", 0); > + else > + MMDRV_Install(drvA, drvA, FALSE); > > HeapFree(GetProcessHeap(), 0, drvA); > PropVariantClear(&pv); > -- > 1.7.11.3 Shouldn't this be part of the winepulse patch itself? It will never get used in Wine, as is... -- -Austin

2 1

Re: d3dx9_36 [patch 1/3]: Fix the case out = in for D3DXSHRotateZ
by Nozomi Kodama Oct. 17, 2012

Oct. 17, 2012

My bad. My patch pass the tests but not yours. Can you check these facts? Nozomi >Hi Nozomi, > >the huge arrays, the loop, it looks a bit ugly. Your implementation makes the D3DXSHRotateZ function a >bit more compatible. What comes into my mind is: >D3DXSHRotateZ(out, y, 1,25f, in); >D3DXSHRotateZ(out, y, 1,25f, out); >D3DXSHRotateZ(out, y, 1,25f, &out[x]); // x ... 1 - y * y >D3DXSHRotateZ(&out[x], y, 1,25f, out); // x ... 1 - y * y > >Do the last 2 also work and are they producing the same values as native? > >Attached is a simple hack to succeed your test, it has some major stuff which needs to be fixed first, but >nevertheless, please have a look. I think the implementation could be a lot smaller and faster. What do you >think? I'd like to hear your opinion, before you try to send a patch. >+FLOAT * WINAPI D3DXSHRotateZ(FLOAT *out, UINT order, FLOAT angle, CONST FLOAT *in) >There is a double space after "order,". > >Cheers >Rico

2 2

Re: (try 3) programs/winemenubuilder: scale 64x64 classic icons to 48x48 for Icns format
by Ken Thomases Oct. 17, 2012

Oct. 17, 2012

On Oct 17, 2012, at 9:50 AM, Aric Stewart wrote: > - if (iconDirEntries[i].wBitCount >= best[slot].maxBits) > + if (((best[slot].maxBits && !scaled) || (!best[slot].maxBits) || (scaled && best[slot].scaled)) && (iconDirEntries[i].wBitCount >= best[slot].maxBits || (!scaled && best[slot].scaled))) This could be more clearly written as: if ((!scaled || !best[slot].maxBits || best[slot].scaled) && (iconDirEntries[i].wBitCount >= best[slot].maxBits || (!scaled && best[slot].scaled))) or even: if (scaled && best[slot].maxBits && !best[slot].scaled) continue; /* don't replace unscaled with scaled */ if (iconDirEntries[i].wBitCount >= best[slot].maxBits || (!scaled && best[slot].scaled)) That is, your left side of the outer && has redundancies and all the negatives make it hard to penetrate. -Ken

1 0

Re: [PATCH 4/6] [cmd] Add for /f parsingkeyword parsing, add support for skip=
by Alexandre Julliard Oct. 17, 2012

Oct. 17, 2012

Ann and Jason Edmeades <jason(a)edmeades.me.uk> writes: > Parse the options you can provide with for /f to control the > subsequent parsing, and add support for the simplest (skip). > @@ -1546,6 +1545,13 @@ void WCMD_for (WCHAR *p, CMD_LIST **cmdList) { > int itemNum; > CMD_LIST *thisCmdStart; > int parameterNo = 0; > + WCHAR forf_eol=0; > + int forf_skip=0; > + WCHAR forf_delims[256]; > + WCHAR forf_tokens[MAXSTRING]; > + BOOL forf_usebackq = FALSE; > + static const WCHAR forf_defaultdelims[] = {' ', '\t'}; > + static const WCHAR forf_defaulttokens[] = {'1'}; It's clearly time to start splitting WCMD_for into several helper functions. -- Alexandre Julliard julliard(a)winehq.org

2 1

[PATCH] krnl386.exe: Implement INT 33h function 15h
by Christian Costa Oct. 17, 2012

Oct. 17, 2012

1 0

Re: [PATCH 20/25] mciseq: Check stop condition after sleeping. (Try 5)
by Marcus Meissner Oct. 17, 2012

Oct. 17, 2012

On Wed, Oct 17, 2012 at 10:36:44AM +0200, Joerg-Cyril.Hoehle(a)t-systems.com wrote: > Please note that the patch queue failed to detect some superseded patches: > > 90588: I should not have named that 17/25 in the first place. > 90672 + 90674: the original 20-21/25 were never marked superseded by later patches. > 90841+ 90482: try 3 was superseded by try 4, now superseded by this try 5. I would suggest to resubmit your queue in smaller batches (like eg. 5 pieces per day) and wait for it to get applied. In the current state testbot will not be run on the sequence I think. Ciao, Marcus

1 0

Re: testbot/lib: If we cannot open the log file, then log to stderr.
by Christian Costa Oct. 17, 2012

Oct. 17, 2012

Hi François, While your are at it. I would like to do a suggestion. Indeed that would be great to be able to download some files along with the test. Typically for me avi files and dlls (or .drv) would be great ! :D Thanks Christian, 2012/10/15 Francois Gouget <fgouget(a)codeweavers.com> > --- > testbot/lib/WineTestBot/Log.pm | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/testbot/lib/WineTestBot/Log.pm > b/testbot/lib/WineTestBot/Log.pm > index 6883968..7de6dcb 100644 > --- a/testbot/lib/WineTestBot/Log.pm > +++ b/testbot/lib/WineTestBot/Log.pm > @@ -38,20 +38,18 @@ sub LogMsg(@) > if (!defined $logfile) > { > my $oldumask = umask(002); > - if (open($logfile, ">>", "$LogDir/log")) > - { > - # Flush after each print > - my $tmp=select($logfile); > - $| = 1; > - select($tmp); > - } > - else > + if (!open($logfile, ">>", "$LogDir/log")) > { > require File::Basename; > print STDERR File::Basename::basename($0), ":warning: could not > open '$LogDir/log' for writing: $!\n"; > - $logfile = undef; > + open($logfile, ">>&=", 2); > } > umask($oldumask); > + > + # Flush after each print > + my $tmp=select($logfile); > + $| = 1; > + select($tmp); > } > print $logfile scalar localtime, " ", @_ if ($logfile); > } > -- > 1.7.10.4 > > > >

4 10

Re: [PATCH 3/3] mshtml: Added more ActiveX loading and script binding tests
by Marvin Oct. 17, 2012

Oct. 17, 2012

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

1 0

Tr : d3dx9_36 [patch 1/3]: Fix the case out = in for D3DXSHRotateZ
by Nozomi Kodama Oct. 17, 2012

Oct. 17, 2012

Hi Both your patch and mine fail to pass the attached tests. More thoughts are needed. Nozomi >Hi Nozomi, > >the huge arrays, the loop, it looks a bit ugly. Your implementation makes the D3DXSHRotateZ function a >bit more compatible. What comes into my mind is: >D3DXSHRotateZ(out, y, 1,25f, in); >D3DXSHRotateZ(out, y, 1,25f, out); >D3DXSHRotateZ(out, y, 1,25f, &out[x]); // x ... 1 - y * y >D3DXSHRotateZ(&out[x], y, 1,25f, out); // x ... 1 - y * y > >Do the last 2 also work and are they producing the same values as native? > >Attached is a simple hack to succeed your test, it has some major stuff which needs to be fixed first, but >nevertheless, please have a look. I think the implementation could be a lot smaller and faster. What do you >think? I'd like to hear your opinion, before you try to send a patch. > >+FLOAT * WINAPI D3DXSHRotateZ(FLOAT *out, UINT order, FLOAT angle, CONST FLOAT *in) >There is a double space after "order,". > >Cheers >Rico

1 0

Re: [PATCH 5/5] ddraw/tests: Add some display mode set / restore tests with multiple ddraw objects.
by Marvin Oct. 16, 2012

Oct. 16, 2012

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=22275 Your paranoid android. === WNT4WSSP6 (32 bit ddraw2) === ddraw2.c:2526: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2543: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2551: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2553: Test failed: Got unexpected screen width 800. ddraw2.c:2555: Test failed: Got unexpected screen height 600. ddraw2.c:2574: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2582: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2584: Test failed: Got unexpected screen width 800. ddraw2.c:2586: Test failed: Got unexpected screen height 600. ddraw2.c:2606: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2608: Test failed: Got unexpected screen width 800. ddraw2.c:2610: Test failed: Got unexpected screen height 600. ddraw2.c:2615: Test failed: Got unexpected screen width 800. ddraw2.c:2617: Test failed: Got unexpected screen height 600. ddraw2.c:2630: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2638: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw2.c:2640: Test failed: Got unexpected screen width 800. ddraw2.c:2642: Test failed: Got unexpected screen height 600. ddraw2.c:2650: Test failed: Got unexpected screen width 800. ddraw2.c:2652: Test failed: Got unexpected screen height 600. ddraw2.c:2664: Test failed: SetDipslayMode failed, hr 0x887600e1. === WNT4WSSP6 (32 bit ddraw1) === ddraw1.c:2389: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2406: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2414: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2416: Test failed: Got unexpected screen width 800. ddraw1.c:2418: Test failed: Got unexpected screen height 600. ddraw1.c:2437: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2445: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2447: Test failed: Got unexpected screen width 800. ddraw1.c:2449: Test failed: Got unexpected screen height 600. ddraw1.c:2469: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2471: Test failed: Got unexpected screen width 800. ddraw1.c:2473: Test failed: Got unexpected screen height 600. ddraw1.c:2478: Test failed: Got unexpected screen width 800. ddraw1.c:2480: Test failed: Got unexpected screen height 600. ddraw1.c:2493: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2501: Test failed: SetDipslayMode failed, hr 0x887600e1. ddraw1.c:2503: Test failed: Got unexpected screen width 800. ddraw1.c:2505: Test failed: Got unexpected screen height 600. ddraw1.c:2513: Test failed: Got unexpected screen width 800. ddraw1.c:2515: Test failed: Got unexpected screen height 600. ddraw1.c:2527: Test failed: SetDipslayMode failed, hr 0x887600e1.

1 0

Re: [PATCH] user32: Add tests showing that MapWindowPoints, ClientToScreen and ScreenToClient never fail.
by Henri Verbeet Oct. 16, 2012

Oct. 16, 2012

On 16 October 2012 20:12, Christian Costa <titan.costa(a)gmail.com> wrote: > + //SetWindowPos(wnd, NULL, 0, 0, rect.right / 2, rect.bottom / 2, 0); C99 comment. I don't think these are very interesting cases though, it's probably more interesting what happens with e.g. a destroyed window, or a fullscreen window.

3 3

Re: [PATCH] user32: Add tests showing that MapWindowPoints, ClientToScreen and ScreenToClient never fail.
by Austin English Oct. 16, 2012

Oct. 16, 2012

On Tue, Oct 16, 2012 at 11:12 AM, Christian Costa <titan.costa(a)gmail.com> wrote: ... > + wnd = CreateWindowA("static", NULL, 0, 0, 0, rect.right / 2, rect.bottom / 2, 0, 0, 0, 0); > + //SetWindowPos(wnd, NULL, 0, 0, rect.right / 2, rect.bottom / 2, 0); Leftover debug code. Cheers, Austin

2 1

Re: programs/winemenubuilder: scale 64x64 classic icons to 48x48 for Mac OS/X
by Alexandre Julliard Oct. 16, 2012

Oct. 16, 2012

Aric Stewart <aric(a)codeweavers.com> writes: > @@ -403,6 +404,13 @@ static HRESULT convert_to_native_icon(IStream *icoFile, int *indices, int numInd > WINE_ERR("error 0x%08X creating IWICBitmapDecoder\n", hr); > goto end; > } > +#ifdef __APPLE__ > + hr = IWICImagingFactory_CreateBitmapScaler(factory, &scaler); > + if (FAILED(hr)) > + { > + WINE_WARN("error 0x%08X creating IWICBitmapScaler\n", hr); > + } > +#endif Please try to do that without adding #ifdefs in generic functions. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [PATCH] cmd: Properly handle multibyte characters in batch files.
by Alexandre Julliard Oct. 16, 2012

Oct. 16, 2012

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > @@ -244,7 +244,8 @@ WCHAR *WCMD_fgets(WCHAR *buf, DWORD noChars, HANDLE h) > > if (!WCMD_is_console_handle(h) && i != charsRead) { > /* Sets file pointer to the start of the next line, if any */ > - filepos.QuadPart += i + 1 + (buf[i] == '\r' ? 1 : 0); > + INT bytes = WideCharToMultiByte(GetConsoleCP(), 0, buf, i, NULL, 0, NULL, NULL); > + filepos.QuadPart += bytes + 1 + (buf[i] == '\r' ? 1 : 0); There's no guarantee that all characters would round-trip properly, this should be based on the original multibyte length somehow. -- Alexandre Julliard julliard(a)winehq.org

1 0

user32 tests patch
by Christian Costa Oct. 16, 2012

Oct. 16, 2012

Forget my user32 patch. MapWindowPoints tests are wrong.

1 0

Re: [PATCH 2/6] d3dcompiler: Improve a Nvidia GPU recognition fallback.
by Matteo Bruni Oct. 16, 2012

Oct. 16, 2012

Ehh, this patch should have been called something like "wined3d: Improve a Nvidia GPU recognition fallback.", but well...

1 0

Re: [PATCH 7/8] mshtml: Added IHTMLScriptElement::put_src tests
by Marvin Oct. 16, 2012

Oct. 16, 2012

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=22264 Your paranoid android. === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: dwrite: Implement GetMetrics() for IDWriteFont
by Dmitry Timoshkov Oct. 16, 2012

Oct. 16, 2012

Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > + static const WCHAR arialW[] = {'A','r','i','a','l',0}; Please don't use fonts which Wine doesn't provide. Testing Tahoma is fine. -- Dmitry.

2 1