Re: [PATCH 2/2] d3drm/tests: Add QueryInterface tests for IDirect3DRMTexture*.

[Henri Verbeet]

On 18 January 2016 at 19:57, Aaryaman Vasishta
 wrote:
> - char *file = NULL;

Why do you initialize this to NULL?
> - BOOL check;
> -
> -
> - hr = Direct3DRMCreate(&d3drm1);

The extra empty line is weird. Is that intentional?
> - file = create_bitmap(1, 1, TRUE);
> - hr = IDirect3DRM_LoadTexture(d3drm1, file, &texture1);

I think "file" is a bad name. I'd expect that to be a file handle or
descriptor, not a string.
> - ok(SUCCEEDED(hr), "Failed to load texture (hr = %x).\n", hr);
> - ok(SUCCEEDED(hr), "Cannot get IDirect3DRMTexture interface (hr = %x)\n", hr);

This looks like it was leftover from somewhere, but in either case it
doesn't test what the message says it's testing.

It's fairly harmless here, but %x is a bad format in most cases. E.g.,
is "1000" decimal 4096 or decimal 1000? Just use %#x. Ideally you'd
also use the same message format as the other D3D tests.
> - check = DeleteFileA(file);
> - ok(check, "Cannot delete image stored in %s (error = %d).\n", file, GetLastError());
>
> +}

You're leaking "file".