Re: [PATCH v5 0/6] MR8326: ntdll: Implement RtlActivateActivationContextUnsafeFast().

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/kernel32/tests/actctx.c:
> -
>    DeleteFileA("test1.manifest");
> -
>    return;
> - }
> - context1 = test_create("test1.manifest");
> - ok(context1 != INVALID_HANDLE_VALUE, "Failed to create context, error %lu.\n", GetLastError());
> - DeleteFileA("test1.manifest");
> - context2 = test_create("test2.manifest");
> - ok(context2 != INVALID_HANDLE_VALUE, "Failed to create context, error %lu.\n", GetLastError());
> - DeleteFileA("test2.manifest");
> -
> - /* Size and Format don't seem to be checked */
> - frame_extended1.Size = 0;
> - frame_extended1.Format = 0xdeadbeef;
> - frame = pRtlActivateActivationContextUnsafeFast(&frame_extended1, context1);
> - /* RtlActivateActivationContextUnsafeFast() has a different prototype on <= Win7 */
> - ok(frame == &frame_extended1.Frame || broken(frame != &frame_extended1.Frame), "Got unexpected frame.\n");

Still, I would remove this one. Does React Native use return value from this function? If not, I would just ignore it in tests.