Nikolay Sivov (@nsivov) commented about dlls/evr/tests/evr.c:
ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
hr = IMFTransform_GetInputCurrentType(transform, 0, &media_type);
ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
ok(media_type == video_type, "Unexpected media type instance.\n");
IMFMediaType_Release(media_type);
hr = IMFTransform_SetOutputType(transform, 0, media_type2, 0);
ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
hr = IMFTransform_GetOutputCurrentType(transform, 0, &media_type);
ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
ok(media_type2 == media_type, "Unexpected media type instance.\n");
IMFMediaType_Release(media_type2); IMFMediaType_Release(media_type);
/* It shouldn't cause memory exception if clear again */
Please simplify this comment a bit, for example "Resetting type twice".