October 2017 - Wine-devel - list.winehq.org

Re: [PATCH 3/4] mshtml/tests: Skip postMessage test on IE7.
by Marvin 24 Oct '17

24 Oct '17

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 https://testbot.winehq.org/JobDetails.pl?Key=33746 Your paranoid android. === w7pro64 (32 bit htmldoc) === htmldoc.c:3118: Test failed: Incorrect error code: -2146697211 htmldoc.c:3123: Test failed: Page address: L"http://test.winehq.org/tests/winehq_snapshot/" htmldoc.c:5895: Test failed: expected OnChanged_1012 htmldoc.c:5896: Test failed: expected Exec_HTTPEQUIV htmldoc.c:5898: Test failed: expected Exec_SETTITLE htmldoc.c:5939: Test failed: expected FireNavigateComplete2

1 0

Re: [PATCH 2/3] gdiplus: Add internal string format properties used by EMF+ records
by Vincent Povirk 24 Oct '17

24 Oct '17

I think the margins in default_drawstring_format should be 1/6. It's not currently used in GdipDrawString or GdipMeasureString, but that's something we may change in the future. Also, they're copied over to generic_default_format which would've had 1/6 margins previously.

1 0

Re: [01/10] secur32: Implement AcquireCredentialsHandle for Kerberos.
by Alexandre Julliard 24 Oct '17

24 Oct '17

Hans Leidekker <hans(a)codeweavers.com> writes: > Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> > --- > configure.ac | 18 +++ > dlls/secur32/Makefile.in | 2 +- > dlls/secur32/kerberos.c | 276 +++++++++++++++++++++++++++++++++++++++++--- > dlls/secur32/secur32.c | 1 + > dlls/secur32/secur32_priv.h | 1 + > 5 files changed, 282 insertions(+), 16 deletions(-) I liked the idea of putting this in kerberos.dll. Have you given up on this? -- Alexandre Julliard julliard(a)winehq.org

3 8

Re: [PATCH] cmd: Implement builtin mklink command
by Alexandre Julliard 24 Oct '17

24 Oct '17

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > + if(hard && !CreateHardLinkW(file1, file2, NULL)) > + WCMD_output_stderr(WCMD_LoadMessage(WCMD_READFAIL), file1); > + else if(!junction && !CreateSymbolicLinkW(file1, file2, isdir)) > + WCMD_output_stderr(WCMD_LoadMessage(WCMD_READFAIL), file1); > + else if(junction) > + WINE_TRACE("Juction links currently not supported.\n"); Surely the options have to be exclusive. It makes no sense to try to create multiple links for the same file. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] msxml/tests: Add test for \u escape sequences in regular expressions.
by Nikolay Sivov 24 Oct '17

24 Oct '17

On 12.10.2017 22:37, Zebediah Figura wrote: > For bug #43581. > > Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> > --- > dlls/msxml3/tests/schema.c | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/dlls/msxml3/tests/schema.c b/dlls/msxml3/tests/schema.c > index ee84b92158..dc65811adb 100644 > --- a/dlls/msxml3/tests/schema.c > +++ b/dlls/msxml3/tests/schema.c > @@ -407,6 +407,15 @@ static const CHAR szOpenSeqXML2[] = "<test><x/><x/><y/><z/><z/><w/></test>"; > static const CHAR szOpenSeqXML3[] = "<test><w/><x/><x/><y/><z/><z/></test>"; > static const CHAR szOpenSeqXML4[] = "<test><x/><x/><y/><z/><z/><v/></test>"; > > +static const CHAR szUnicodeEscapeXDR[] = > +"<schema xmlns='http://www.w3.org/2001/XMLSchema'>" > +" <simpleType name='type1'>" > +" <restriction base='string'>" > +" <pattern value='\\u0020'/>" > +" </restriction>" > +" </simpleType>" > +"</schema>"; > + > #define check_ref_expr(expr, n) { \ > LONG ref = expr; \ > ok(ref == n, "expected %i refs, got %i\n", n, ref); \ > @@ -1154,6 +1163,23 @@ static void test_XDR_schemas(void) Is it XDR "feature" only, that they didn't add to XSDs? > IXMLDOMDocument2_Release(schema); > IXMLDOMSchemaCollection_Release(cache); > > + /* test for \u escapes */ > + schema = create_document_version(60, &IID_IXMLDOMDocument2); > + cache = create_cache_version(60, &IID_IXMLDOMSchemaCollection); > + if (!schema || !cache) > + { > + if (schema) IXMLDOMDocument2_Release(schema); > + if (cache) IXMLDOMSchemaCollection_Release(cache); > + return; > + } Is this really specific to msxml6? It's better to test on all of them to get a full picture, but probably all of them support this extension if 6th does. If you want to skip on unsupported class, just return after first failure, no need to check both pointers. > + > + ole_check(IXMLDOMDocument2_loadXML(schema, _bstr_(szUnicodeEscapeXDR), &b)); > + ok(b == VARIANT_TRUE, "failed to load XML string\n"); > + todo_wine ole_check(IXMLDOMSchemaCollection_add(cache, _bstr_(""), _variantdoc_(schema))); I think it's better to get rid of ole_check(), but it's up to you. > + > + IXMLDOMDocument2_Release(schema); > + IXMLDOMSchemaCollection_Release(cache); > + > free_bstrs(); > } > >

2 1

Re: oleaut32: Add ARM support to DispCallFunc().
by André Hentschel 23 Oct '17

23 Oct '17

Hi Donna and Welcome to the Wine Project! Whats your motivation for this patch? You did a very good job for a firsttimer! Some more notes below: Am 22.10.2017 um 03:04 schrieb Donna Whisnant: > Adds ARM ABI support to DispCallFunc() to allow IDispatch invoke calls > to succeed on ARM platforms. This change specifically targets only > 32-bit little-endian (ARMEL) platform CPUs. It's believed to likely be > compatible with 64-bit and/or big-endian (ARMEB) platforms, but testing > for those other platforms should be completed before enabling. > > Tested on Raspbian Stretch 2017-09-07 RPi image using a Qemu systemd > container to compile and run it on an x86-64 Host.. When running on > Raspberry Pi hardware, a 3G/1G split kernel needs to be used for Wine > to function. > > Signed-off-by: Donna Whisnant <dewhisna(a)dewtronics.com> > --- > dlls/oleaut32/typelib.c | 221 +++++++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 219 insertions(+), 2 deletions(-) > > diff --git a/dlls/oleaut32/typelib.c b/dlls/oleaut32/typelib.c > index ebf6d85..d7ef212 100644 > --- a/dlls/oleaut32/typelib.c > +++ b/dlls/oleaut32/typelib.c > @@ -6302,8 +6302,48 @@ static HRESULT WINAPI ITypeInfo_fnGetIDsOfNames( ITypeInfo2 *iface, > return DISP_E_UNKNOWNNAME; > } > > +#if defined(__arm__) && defined(__ARM_32BIT_STATE) && !defined(__ARMEB__) > +/* Note: this may work OK on 64-bit and/or ARMEB, but has currently only been tested for 32-bit ARMEL */ While i doubt there is something risky here regarding endianess you can keep the ARMEB check, but get rid of the 32bit check, it might be about Thunk(16bit) vs. ARM(32bit) but I'm not sure. Anyway, this won't work on 64bit, because the instruction set is totally different, but it inspires me to do the 64bit version as soon this lands upstream ;) > > -#ifdef __i386__ > +extern LONGLONG call_method( void *func, int nb_stk_args, const DWORD *stk_args, const DWORD *reg_args ); > +__ASM_GLOBAL_FUNC( call_method, > + /* r0 = *func > + * r1 = nb_stk_args > + * r2 = *stk_args (pointer to 'nb_stk_args' DWORD values to push on stack) > + * r3 = *reg_args (pointer to 8, 64-bit d0-d7 (double) values OR as 16, 32-bit s0-s15 (float) values, followed by 4, 32-bit (DWORD) r0-r3 values) > + */ > + > + "push {fp, lr}\n\t" /* Save frame pointer and return address (stack still aligned to 8 bytes) */ > + __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") > + __ASM_CFI(".cfi_rel_offset %fp,0\n\t") > + "add fp, sp, #4\n\t" /* Setup frame pointer */ Is this adjusted FP used somewhere? (I forgot a lot of things regarding arm32...) > + __ASM_CFI(".cfi_def_cfa_register %fp\n\t") > + > + "lsls r1, r1, #2\n\t" /* r1 = nb_stk_args * sizeof(DWORD) */ > + "beq 1f\n\t" /* Skip allocation if no stack args */ > + "add r2, r2, r1\n" /* Calculate ending address of incoming stack data */ > + "2:\tldr ip, [r2, #-4]!\n\t" /* Get next value */ > + "str ip, [sp, #-4]!\n\t" /* Push it on the stack */ > + "subs r1, r1, #4\n\t" /* Decrement count */ > + "bgt 2b\n\t" /* Loop till done */ > + > + "1:\tvldm r3!, {s0-s15}\n\t" /* Load the s0-s15/d0-d7 arguments */ > + "mov ip, r0\n\t" /* Save the function call address to ip before we nuke r0 with arguments to pass */ > + "ldm r3, {r0-r3}\n\t" /* Load the r0-r3 arguments */ > + > + "blx ip\n\t" /* Call the target function */ > + > + "sub sp, fp, #4\n\t" /* Clean the stack using 'fp' to allow for either stdcall or cdecl calls */ > + __ASM_CFI(".cfi_def_cfa %sp,4\n\t") > + "pop {fp, pc}\n\t" /* Restore FP and return */ > + __ASM_CFI(".cfi_same_value %fp\n\t") /* Here for completeness and symmetry with other cpus, but won't be reached */ > + ) > + > +/* same function but returning single/double floating point */ > +static float (* const call_float_method)(void *, int, const DWORD *, const DWORD *) = (void *)call_method; > +static double (* const call_double_method)(void *, int, const DWORD *, const DWORD *) = (void *)call_method; > + > +#elif defined(__i386__) > > extern LONGLONG call_method( void *func, int nb_args, const DWORD *args, int *stack_offset ); > __ASM_GLOBAL_FUNC( call_method, > @@ -6637,7 +6677,184 @@ DispCallFunc( > void* pvInstance, ULONG_PTR oVft, CALLCONV cc, VARTYPE vtReturn, UINT cActuals, > VARTYPE* prgvt, VARIANTARG** prgpvarg, VARIANT* pvargResult) > { > -#ifdef __i386__ > +#if defined(__arm__) && defined(__ARM_32BIT_STATE) && !defined(__ARMEB__) > +/* Note: this may work OK on 64-bit and/or ARMEB, but has currently only been tested for 32-bit ARMEL */ Same as above > + int argspos; > + void *func; > + UINT i; > + DWORD *args; > + struct { > + union { > + float s[16]; > + double d[8]; > + } sd; > + DWORD r[4]; > + } regs; > + int rcount; /* 32-bit register index count */ > + int scount; /* single-precision float register index count (will be incremented twice for doubles, plus alignment) */ > + > + TRACE("(%p, %ld, %d, %d, %d, %p, %p, %p (vt=%d))\n", > + pvInstance, oVft, cc, vtReturn, cActuals, prgvt, prgpvarg, > + pvargResult, V_VT(pvargResult)); You can go up to 100 chars per line, so be free to rearrange this > + > + if (cc != CC_STDCALL && cc != CC_CDECL) > + { > + FIXME("unsupported calling convention %d\n",cc); > + return E_INVALIDARG; > + } > + > + argspos = 0; > + rcount = 0; > + scount = 0; > + > + /* Determine if we need to pass a pointer for the return value as arg 0. If so, do that */ > + /* first as it will need to be in the 'r' registers: */ > + switch (vtReturn) > + { > + case VT_DECIMAL: > + case VT_VARIANT: > + regs.r[rcount++] = (DWORD)pvargResult; /* arg 0 is a pointer to the result */ > + break; > + case VT_HRESULT: > + WARN("invalid return type %u\n", vtReturn); > + return E_INVALIDARG; > + default: /* And all others are in 'r', 's', or 'd' registers or have no return value */ > + break; > + } > + > + if (pvInstance) > + { > + const FARPROC *vtable = *(FARPROC **)pvInstance; > + func = vtable[oVft/sizeof(void *)]; > + regs.r[rcount++] = (DWORD)pvInstance; /* the This pointer is always the first parameter */ > + } > + else func = (void *)oVft; > + > + /* maximum size for an argument is sizeof(VARIANT). Also allow for return pointer and stack alignment. */ > + args = heap_alloc(sizeof(VARIANT) * cActuals + sizeof(DWORD) * 4 ); > + > + for (i = 0; i < cActuals; i++) > + { > + VARIANT *arg = prgpvarg[i]; > + DWORD *pdwarg = (DWORD *)(arg); /* a reinterpret_cast of the variant, used for copying structures when they are split between registers and stack */ > + int ntemp; /* Used for counting words split between registers and stack */ > + > + switch (prgvt[i]) > + { > + case VT_EMPTY: > + break; > + case VT_R4: /* these must be 4-byte aligned, and put in 's' regs or stack, as they are single-floats */ > + if (scount < 16) > + { > + regs.sd.s[scount++] = V_R4(arg); > + } else { > + args[argspos++] = V_UI4(arg); > + } I think no curly brackets are necessary here at all, but we usually don't open them in the same line as the if/else statement for new code > + break; > + case VT_R8: /* these must be 8-byte aligned, and put in 'd' regs or stack, as they are double-floats */ > + case VT_DATE: > + if (scount < 15) > + { > + scount += (scount % 2); /* align scount to next whole double */ > + regs.sd.d[scount/2] = V_R8(arg); > + scount += 2; > + } else { > + scount = 16; /* Make sure we flag that all 's' regs are full */ > + argspos += (argspos % 2); /* align argspos to 8-bytes */ > + memcpy( &args[argspos], &V_R8(arg), sizeof(V_R8(arg)) ); > + argspos += sizeof(V_R8(arg)) / sizeof(DWORD); > + } > + break; > + case VT_I8: /* these must be 8-byte aligned, and put in 'r' regs or stack, as they are long-longs */ > + case VT_UI8: > + case VT_CY: > + if (rcount < 3) > + { > + rcount += (rcount % 2); /* align rcount to 8-byte register pair */ > + memcpy( &regs.r[rcount], &V_UI8(arg), sizeof(V_UI8(arg)) ); > + rcount += sizeof(V_UI8(arg)) / sizeof(DWORD); > + } else { > + rcount = 4; /* Make sure we flag that all 'r' regs are full */ > + argspos += (argspos % 2); /* align argspos to 8-bytes */ > + memcpy( &args[argspos], &V_UI8(arg), sizeof(V_UI8(arg)) ); > + argspos += sizeof(V_UI8(arg)) / sizeof(DWORD); > + } > + break; > + case VT_DECIMAL: /* these structures are 8-byte aligned, and put in 'r' regs or stack, can be split between the two */ > + case VT_VARIANT: > + /* 8-byte align 'r' and/or stack: */ > + if (rcount < 3) > + { > + rcount += (rcount % 2); > + } else { > + rcount = 4; > + argspos += (argspos % 2); > + } > + ntemp = sizeof(*arg) / sizeof(DWORD); > + while (ntemp > 0) > + { > + if (rcount < 4) > + { > + regs.r[rcount++] = *pdwarg++; > + } else { > + args[argspos++] = *pdwarg++; > + } > + --ntemp; > + } > + break; > + case VT_BOOL: /* VT_BOOL is 16-bit but BOOL is 32-bit, needs to be extended */ > + if (rcount < 4) > + { > + regs.r[rcount++] = V_BOOL(arg); > + } else { > + args[argspos++] = V_BOOL(arg); > + } > + break; > + default: > + if (rcount < 4) > + { > + regs.r[rcount++] = V_UI4(arg); > + } else { > + args[argspos++] = V_UI4(arg); > + } > + break; > + } > + TRACE("arg %u: type %s %s\n", i, debugstr_vt(prgvt[i]), debugstr_variant(arg)); > + } > + > + argspos += (argspos % 2); /* Make sure stack function alignment is 8-byte */ > + > + TRACE("rcount: %d, scount: %d, argspos: %d\n", rcount, scount, argspos); > + > + switch (vtReturn) > + { > + case VT_EMPTY: /* EMPTY = no return value */ > + case VT_DECIMAL: /* DECIMAL and VARIANT already have a pointer argument passed (see above) */ > + case VT_VARIANT: > + call_method( func, argspos, args, (DWORD*)&regs ); > + break; > + case VT_R4: > + V_R4(pvargResult) = call_float_method( func, argspos, args, (DWORD*)&regs ); > + break; > + case VT_R8: > + case VT_DATE: > + V_R8(pvargResult) = call_double_method( func, argspos, args, (DWORD*)&regs ); > + break; > + case VT_I8: > + case VT_UI8: > + case VT_CY: > + V_UI8(pvargResult) = call_method(func, argspos, args, (DWORD*)&regs ); > + break; > + default: > + V_UI4(pvargResult) = call_method( func, argspos, args, (DWORD*)&regs ); > + break; > + } > + heap_free( args ); > + if (vtReturn != VT_VARIANT) V_VT(pvargResult) = vtReturn; > + TRACE("retval: %s\n", debugstr_variant(pvargResult)); > + return S_OK; > + > +#elif defined(__i386__) > int argspos, stack_offset; > void *func; > UINT i; >

1 0

Re: [3/4] include: Add twain 2.x header for use in data sources.
by Henri Verbeet 23 Oct '17

23 Oct '17

On 20 October 2017 at 21:16, Vincent Povirk <vincent(a)codeweavers.com> wrote: > diff --git a/include/twain2.h b/include/twain2.h > new file mode 100644 > index 0000000..8455649 > --- /dev/null > +++ b/include/twain2.h > @@ -0,0 +1,2131 @@ > +/* ======================================================================== *\ > + > + Copyright (C) 2007 TWAIN Working Group: Adobe Systems Incorporated, > + AnyDoc Software Inc., Eastman Kodak Company, Fujitsu Computer Products > + of America, JFL Peripheral Solutions Inc., Ricoh Corporation, and > + Xerox Corporation. All rights reserved. > + > + Copyright (C) 1991, 1992 TWAIN Working Group: Aldus, Caere, Eastman-Kodak, > + Hewlett-Packard and Logitech Corporations. All rights reserved. > + > + Copyright (C) 1997 TWAIN Working Group: Bell+Howell, Canon, DocuMagix, > + Fujitsu, Genoa Technology, Hewlett-Packard, Kofax Imaging Products, and > + Ricoh Corporation. All rights reserved. > + > + Copyright (C) 1998 TWAIN Working Group: Adobe Systems Incorporated, > + Canon Information Systems, Eastman Kodak Company, > + Fujitsu Computer Products of America, Genoa Technology, > + Hewlett-Packard Company, Intel Corporation, Kofax Image Products, > + JFL Peripheral Solutions Inc., Ricoh Corporation, and Xerox Corporation. > + All rights reserved. > + > + Copyright (C) 2000 TWAIN Working Group: Adobe Systems Incorporated, > + Canon Information Systems, Digimarc Corporation, Eastman Kodak Company, > + Fujitsu Computer Products of America, Hewlett-Packard Company, > + JFL Peripheral Solutions Inc., Ricoh Corporation, and Xerox Corporation. > + All rights reserved. > + Doesn't the above essentially say we can't distribute this?

3 5

Re: [v2 PATCH 1/2] gdiplus/metafile: Implement EmfPlusRecordTypeSetClipRegion playback
by Vincent Povirk 23 Oct '17

23 Oct '17

Is there a reason to clone the region instead of using it directly from the object table?

1 1

Re: [PATCH] ddraw/tests: Avoid dereferencing a null pointer in case of failure.
by Henri Verbeet 23 Oct '17

23 Oct '17

On 22 October 2017 at 00:32, Zebediah Figura <z.figura12(a)gmail.com> wrote: > This should fix a persistent crash on one of the XP test machines. > > Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> > --- Why does it fail? It shouldn't. Does this run against Microsoft's ddraw, or against some VM replacement?

2 2

Re: [PATCH] msvcrt: Implement _memicmp_l()
by Marvin 23 Oct '17

23 Oct '17

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 https://testbot.winehq.org/JobDetails.pl?Key=33714 Your paranoid android. === wvistau64 (32 bit string) === string.c:178: Test failed: errno = 0, expected EINVAL string.c:185: Test failed: errno = 0, expected EINVAL string.c:192: Test failed: errno = 0, expected EINVAL string.c:221: Test failed: errno = 0, expected EINVAL string.c:228: Test failed: errno = 0, expected EINVAL string.c:235: Test failed: errno = 0, expected EINVAL === w7u (32 bit string) === string.c:178: Test failed: errno = 0, expected EINVAL string.c:185: Test failed: errno = 0, expected EINVAL string.c:192: Test failed: errno = 0, expected EINVAL string.c:221: Test failed: errno = 0, expected EINVAL string.c:228: Test failed: errno = 0, expected EINVAL string.c:235: Test failed: errno = 0, expected EINVAL === w1064 (32 bit string) === string.c:178: Test failed: errno = 0, expected EINVAL string.c:185: Test failed: errno = 0, expected EINVAL string.c:192: Test failed: errno = 0, expected EINVAL string.c:221: Test failed: errno = 0, expected EINVAL string.c:228: Test failed: errno = 0, expected EINVAL string.c:235: Test failed: errno = 0, expected EINVAL === wvistau64 (64 bit string) === string.c:178: Test failed: errno = 0, expected EINVAL string.c:185: Test failed: errno = 0, expected EINVAL string.c:192: Test failed: errno = 0, expected EINVAL string.c:221: Test failed: errno = 0, expected EINVAL string.c:228: Test failed: errno = 0, expected EINVAL string.c:235: Test failed: errno = 0, expected EINVAL === w1064 (64 bit string) === string.c:178: Test failed: errno = 0, expected EINVAL string.c:185: Test failed: errno = 0, expected EINVAL string.c:192: Test failed: errno = 0, expected EINVAL string.c:221: Test failed: errno = 0, expected EINVAL string.c:228: Test failed: errno = 0, expected EINVAL string.c:235: Test failed: errno = 0, expected EINVAL

1 0