Re: [PATCH v2 0/5] MR8326: ntdll: Implement RtlActivateActivationContextUnsafeFast().

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/kernel32/tests/actctx.c:
>  ReleaseActCtx( context );
>
> }
>
> +static void test_RtlActivateActivationContextUnsafeFast(void)
> +{
> - RTL_CALLER_ALLOCATED_ACTIVATION_CONTEXT_STACK_FRAME_EXTENDED frame_extended1 = {0}, frame_extended2 = {0};
> - HANDLE context1, context2, current_context;
> - RTL_ACTIVATION_CONTEXT_STACK_FRAME *frame;
> - BOOL ret;
> -
> - /* RtlActivateActivationContextUnsafeFast() has a different prototype on <= Win7 */
> - if (!is_win8_plus())
> - {
> -
>    skip("Test requires Win8+.\n");
> -
>    return;
> - }

This will skip on Wine and we don't need to do that. How different is it? If it's only about return value I would still use current prototype, but adjusted tests to use frame_extended.Frame instead of return value, if it's the same value.