[PATCH v2] oleaut32/tests: Mark some tmarshal tests as todo on x64 and ARM
André Hentschel
nerv at dawncrow.de
Tue Feb 13 07:02:04 CST 2018
Am 13.02.2018 um 09:38 schrieb Huw Davies:
> On Mon, Feb 12, 2018 at 07:48:59PM -0700, Alex Henrie wrote:
>> Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
>> ---
>> dlls/oleaut32/tests/tmarshal.c | 68 ++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 68 insertions(+)
>>
>> diff --git a/dlls/oleaut32/tests/tmarshal.c b/dlls/oleaut32/tests/tmarshal.c
>> index 67b8e22642..b716461bd6 100644
>> --- a/dlls/oleaut32/tests/tmarshal.c
>> +++ b/dlls/oleaut32/tests/tmarshal.c
>> @@ -1305,6 +1305,9 @@ static void test_typelibmarshal(void)
>>
>> IStream_Seek(pStream, ullZero, STREAM_SEEK_SET, NULL);
>> hr = CoUnmarshalInterface(pStream, &IID_IKindaEnumWidget, (void **)&pKEW);
>> +#ifndef __i386__
>> + todo_wine
>> +#endif
>> ok_ole_success(hr, CoUnmarshalInterface);
>> IStream_Release(pStream);
>>
>
> Scattering #ifdefs throughout the code looks pretty ugly.
> How about:
>
> #ifdef __i386__
> static const int tmarshal_todo = 0;
> #else
> static const int tmarshal_todo = 1;
> #endif
>
> then
>
> todo_wine_if(tmarshal_todo)
> ok_ole_success(hr, CoUnmarshalInterface);
>
> Huw.
>
>
+1
More information about the wine-devel
mailing list