Rémi Bernon (@rbernon) commented about dlls/mf/tests/transform.c:
ULONG ret, i; HRESULT hr;
- winetest_push_context("wmvdec");
- winetest_push_context("WMV decoder");
I think it would be better to keep these context string as short as possible. They tend to quickly add up and cause output overflows. What about `wmvdmod`?
Same with other context strings above, I think it'd be better to make them shorter (ie: `in %lu` / `out %lu` instead of `GetInputType` / `GetOutputType` + `Type %lu`).
If you think it's worth having more context I'd suggest changing the individual test messages to something more informative than `Got unexpected hr %#lx`. For instance `GetInputType returned %#lx` or something, although I'm not completely convinced it's really necessary.