Re: [PATCH v9 0/1] MR766: advapi32/test: Add tests for RegLoadAppKey.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/advapi32/tests/registry.c:
> - RegCloseKey(appkey);
> -
> - wait_file_available(hivefilepath);
> -
> - appkey = NULL;
> - ret = RegLoadAppKeyA(hivefilepath, &appkey, KEY_READ, 0, 0);
> - ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %ld\n", ret);
> - ok(appkey != NULL, "got a null key\n");
> -
> - size = sizeof(test_data);
> - memset(output, 0xff, sizeof(output));
> - ret = RegGetValueA(appkey, NULL, "testkey", RRF_RT_REG_BINARY, NULL, output, &size);
> - todo_wine ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %ld\n", ret);
> - ok(size == sizeof(test_data), "size doesn't match %ld != %ld\n", size, (DWORD)sizeof(test_data));
> - for (int i = 0; i < sizeof(test_data); ++i)
> -
>    todo_wine ok(test_data[i] == output[i], "output is not what expected i=%d %d != %d\n", i, test_data[i], output[i]);

Using `memcmp()` here would be simpler.