Re: [PATCH 0/5] MR7715: Implement dynamic formats in directdraw media stream.

[Anton Baskanov (＠baskanov)]

Anton Baskanov (@baskanov) commented about dlls/amstream/tests/amstream.c:
> - props.cBuffers = 0;
> - props.cbPrefix = 0;
> - hr = IMemAllocator_SetProperties(new_allocator, &props, &props2);
> - ok(hr == S_OK, "Got hr %#lx.\n", hr);
> -
> - hr = IMemInputPin_NotifyAllocator(mem_input, new_allocator, TRUE);
> - ok(hr == S_OK, "Got hr %#lx.\n", hr);
> -
> - mem_allocator = NULL;
> - hr = IMemInputPin_GetAllocator(mem_input, &mem_allocator);
> - ok(hr == S_OK, "Got hr %#lx.\n", hr);
> - todo_wine ok (mem_allocator == ddraw_allocator, "Expected GetAllocator to return ddraw allocator.\n");
> - todo_wine check_interface(mem_allocator, &IID_IDirectDrawMediaStream, TRUE);
> - IMemAllocator_Release(mem_allocator);
> -
> - IMemAllocator_Release(new_allocator);

You should check the returned value to make sure there are no reference leaks like it's done in the other tests.