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

Re: fusion: Don't assume a specific compiler behaviour
by Ruslan Kabatsayev July 19, 2012

July 19, 2012

Hello, > Most likely it's rotated on x86 and hardly shifted on ARM, damn compilers. No, it's not compilers, it's intel's "shr" instruction behavior: it masks bit count lower 5 bits, thus making 33 effectively 1. Corresponding ARM instruction most likely does true 33 bit shift giving you zero result. Regards, Ruslan PS I didn't look at types your patch works on, but be careful to not mess up 64 bit types this way. On Thu, Jul 19, 2012 at 11:49 PM, André Hentschel <nerv(a)dawncrow.de> wrote: > maybe it's some braindamage by me, but when i is greater or equal to 32 a shift of HighPart by e.g. 33 doesn't make much sense. For some reason this works on x86 but not on ARM. Most likely it's rotated on x86 and hardly shifted on ARM, damn compilers. > If i'm right i don't want to know how much places there are in Wine with assumptions like this, i'll most likely find them with the testsuite on ARM, but i's quite hard to get to the point where you see them... > --- > dlls/fusion/assembly.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/dlls/fusion/assembly.c b/dlls/fusion/assembly.c > index d04ba73..8e6bfcb 100644 > --- a/dlls/fusion/assembly.c > +++ b/dlls/fusion/assembly.c > @@ -506,7 +506,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 0; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > assembly->numtables++; > } > @@ -522,7 +522,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 0; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > assembly->tables[i].rows = assembly->numrows[offidx]; > offidx++; > @@ -534,7 +534,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 1; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > currofs += get_table_size(assembly, previ) * assembly->numrows[offidx - 1]; > assembly->tables[i].offset = currofs; > -- > 1.7.4.1 > > > -- > > Best Regards, André Hentschel > > >

2 1

Re: wininet: Support ICU_ENCODE_PERCENT, ICU_ENCODE_SPACES_ONLY, and ICU_NO_META.
by Austin English July 19, 2012

July 19, 2012

On Wed, Jul 18, 2012 at 6:33 PM, Alex Henrie <alexhenrie24(a)gmail.com> wrote: > Without this patch, Internet Explorer 6 produces a lot of messages like > "fixme:wininet:InternetCanonicalizeUrlW Unhandled flags 0x04000000" > > Wine's wininet.h was missing ICU_ENCODE_PERCENT, which is present in > the wininet.h from the Windows 7 SDK. > > Wine previously treated ICU_ESCAPE like ICU_DECODE. This is incorrect, > MSDN and empirical tests agree that ICU_ESCAPE should be ignored: > http://msdn.microsoft.com/en-us/library/windows/desktop/aa384342(v=vs.85).a… > https://testbot.winehq.org/JobDetails.pl?Key=20155 Why didn't you include the tests in the patch? -- -Austin

2 2

Re: [1/2]usp10: cache OUTLINETEXTMETRICW instead of just TEXTMETRICW
by Dmitry Timoshkov July 19, 2012

July 19, 2012

Aric Stewart <aric(a)codeweavers.com> wrote: > + sc->tm = heap_alloc(size); > + GetOutlineTextMetricsW(hdc, size, sc->tm); otmSize exists for a reason, if GetOutlineTextMetrics currently doesn't check it this doesn't mean it won't at some point in future. -- Dmitry.

1 0

Re: [PATCH 4/4] ieframe: Avoid useless QI(IID_IWebBrowser2) calls in tests by passing IWebBrowser2 interface where possible
by Marvin July 19, 2012

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

1 0

Re: include/winhttp.h: Make strings in WINHTTP_PROXY_INFO non-const
by Hans Leidekker July 19, 2012

July 19, 2012

On Thu, 2012-07-19 at 07:27 +0000, Heiko Hund wrote: > The strings shouldn't be const according to the docs at [1] and > declaring them as LPWSTR removes some false warnings when > compiling code GlobalFree()ing them with mingw. Please also remove unnecessary casts from Wine.

1 0

Re: [PATCH 01/10] dsound/tests: Add IDirectSound refcount and COM aggregation tests.
by Marvin July 18, 2012

July 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=20152 Your paranoid android. === WNT4WSSP6 (32 bit dsound) === dsound.c:194: Test failed: DirectMusicPerformance create failed: 80070057, expected CLASS_E_NOAGGREGATION dsound.c:196: Test failed: dso = DEADBEEF

1 0

Re: ole32: Forward BSTR functions to oleaut32
by Alexandre Julliard July 18, 2012

July 18, 2012

Jacek Caban <jacek(a)codeweavers.com> writes: > @@ -1244,6 +1244,9 @@ static void COMPOBJ_InitProcess( void ) > static void COMPOBJ_UninitProcess( void ) > { > UnregisterClassW(wszAptWinClass, hProxyDll); > + > + if(oleaut32_instance) > + FreeLibrary(oleaut32_instance); Since oleaut32 imports ole32 you'll never get here. Using a delayed import would probably be easier. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: mshtml: Set outgoing pass-by-address variable in an error case case within confirm_safety_load.
by Jacek Caban July 18, 2012

July 18, 2012

Hi Gerald, On 03/07/12 23:35, Gerald Pfeifer wrote: > I noticed we return in this case, without initializing this variable. > Visual inspection indicates we do not seem to access the variable in > this error case, but a) better safe than sorry, and b) GCC 4.8 currently > warns about it. > > Not sure URLPOLICY_DISALLOW is the best return value, open for better > alternatives. I generally don't think this patch makes things any better (or worse). If we return an error, the caller should not expect this value to be sane. What's the GCC 4.8 warning? Jacek

2 4

Re: d3dx9_36: Implementation of D3DXSHRotateZ
by Matteo Bruni July 18, 2012

July 18, 2012

2012/7/18 Nozomi Kodama <nozomi.kodama(a)yahoo.com>: > Minor nitpicks, but nevertheless: > -@ stub D3DXSHRotateZ(ptr long long ptr) > +@ stdcall D3DXSHRotateZ(ptr long long ptr) ... > +FLOAT* WINAPI D3DXSHRotateZ(FLOAT *out, UINT order, FLOAT angle, CONST FLOAT *in) I guess using "float" in the spec file for the third parameter would be more appropriate (yes, it was already wrong but that's not an excuse ;) + if ( i >= order * order || ( (order > D3DXSH_MAXORDER) && ( i >= D3DXSH_MAXORDER * D3DXSH_MAXORDER ) ) ) + expected = ( i + 1.0f ) * ( i + 1.0f ); Bad indentation. In general the inner for loop in the tests feels awkward, try to rewrite it to avoid the duplicated stuff. + } + } + } You're adding a tab character here, please use only whitespaces for indentation. That said, thanks for your work on d3dx9!

1 0

Re: [PATCH 2/2] vbscript: Added LCase implementation
by Marvin July 18, 2012

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

1 0