[PATCH 2/2] include: Add debug utilities for Media Foundation modules

Alexandre Julliard julliard at winehq.org
Mon Feb 7 09:57:31 CST 2022


Nikolay Sivov <nsivov at codeweavers.com> writes:

> On 2/7/22 17:49, Andrew Eikum wrote:
>> @@ -853,7 +854,7 @@ static HRESULT WINAPI video_mixer_transform_SetInputType(IMFTransform *iface, DW
>>       unsigned int count;
>>       GUID *guids;
>>   -    TRACE("%p, %lu, %p, %#lx.\n", iface, id, media_type, flags);
>> +    TRACE("%p, %lu, %p %s, %#lx.\n", iface, id, media_type, wine_dbgstr_mfmediatype(media_type), flags);
> I haven't tried it, but I agree that it looks useful. One minor thing
> is that on null argument you get two nulls printed, but that's not a
> big deal.

It may be useful, but it looks awfully specific for a global header.  If
we want that sort of thing it would have to be made more generally
useful.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list