Re: [PATCH v14 0/2] MR2778: mfmediaengine: Add support for video effects.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> -
> - /* Wait for MediaEngine to be ready. */
> - res = WaitForSingleObject(notify->ready_event, 5000);
> - ok(!res, "Unexpected res %#lx.\n", res);
> -
> - /* Wait for another update. This makes MediaEngine shutdown more consistent on Windows. */
> - res = WaitForSingleObject(notify->ready_event, 500);
> - /* Timeupdates are missing in Wine. */
> - todo_wine ok(!res, "Unexpected res %#lx.\n", res);
> -
> - SetRect(&dst_rect, 0, 0, desc.Width, desc.Height);
> - hr = IMFMediaEngineEx_TransferVideoFrame(notify->media_engine, (IUnknown *)texture, NULL, &dst_rect, NULL);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
> - ok(video_effect->processing_count, "Unexpected delta %lu.\n", video_effect->processing_count);
> - ok(video_effect2->processing_count, "Unexpected delta %lu.\n", video_effect2->processing_count);

Should text message be reworded? Not clear why that would be a delta. Also we'll probably want '> 0' check for these signed counters.