[PATCH] kernel32/tests/fiber: Add tests for fiber-local storage.
André Hentschel
nerv at dawncrow.de
Sun Jul 10 12:51:58 CDT 2016
Am 10.07.2016 um 19:38 schrieb Nikolay Sivov:
> On 10.07.2016 20:31, André Hentschel wrote:
>> Hi,
>>
>> comments inline
>>
>> Am 08.07.2016 um 00:16 schrieb John Sheu:
>>> +static void test_FiberLocalStorageCallback(PFLS_CALLBACK_FUNCTION cbfunc) {
>>> + DWORD fls;
>>> + BOOL ret;
>>> + PVOID val, val2;
>>> +
>>> + if (!pFlsAlloc || !pFlsSetValue || !pFlsGetValue || !pFlsFree)
>>> + {
>>> + win_skip( "Fiber Local Storage not supported\n" );
>>> + return;
>>> + }
>>> +
>>> + /* Test that the callback is executed */
>>> + SetLastError( 0xdeadbeef );
>>> + cbCount = 0;
>>> + fls = pFlsAlloc( cbfunc );
>>> + ok( fls != FLS_OUT_OF_INDEXES, "FlsAlloc failed with error %u\n", GetLastError() );
>>> +
>>> + SetLastError( 0xdeadbeef );
>>> + val = (PVOID) 0x1587;
>>> + fls_value_to_set = val;
>>> + ret = pFlsSetValue( fls, val );
>>> + ok(ret, "FlsSetValue failed with error %u\n", GetLastError() );
>>> +
>>> + val2 = pFlsGetValue( fls );
>>> + ok(val == val2, "FlsGetValue returned %p, expected %p\n", val2, val);
>>> +
>>> + SetLastError( 0xdeadbeef );
>>> + ret = pFlsFree( fls );
>>> + ok(ret, "FlsFree failed with error %u\n", GetLastError() );
>>> + todo_wine
>>> + {
>>> + ok( cbCount == 1, "Wrong callback count: %d\n", cbCount );
>>> + }
>>
>>
>> Please make those one-liners, like
>> todo_wine ok(...);
>> and the patch should be fine then
>
> How is this a stopper?
It's not directly a stopper.
But I personally really don't like it that way and it somehow prevents me from a sign-off. And I think I'll add myself as a Maintainer for that file, so the style should fit somewhat.
More information about the wine-devel
mailing list