Re: [PATCH v2 0/7] MR10009: mf/topology_loader: Refactor the topology loader and fix a few test failures.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/mf/main.c:
> +        X(MFMediaType_Default),
> +        X(MFMediaType_Protected),
> +        X(MFVideoFormat_H264_ES),
> +        X(MFMediaType_Perception),
> +        X(MFT_CATEGORY_VIDEO_PROCESSOR),
> +#undef X
> +    };
> +    struct guid_def *ret = NULL;
> +
> +    if (guid)
> +        ret = bsearch(guid, guid_defs, ARRAY_SIZE(guid_defs), sizeof(*guid_defs), debug_compare_guid);
> +
> +    return ret ? wine_dbg_sprintf("%s", ret->name) : wine_dbgstr_guid(guid);
> +}
> +
> +const char *debugstr_media_subtype(IMFMediaType *media_type)
> Why subtype? What about simply `debugstr_media_type` instead?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10009#note_133374