Re: [PATCH 0/3] MR2247: mf/tests: Test ProcessInput and ProcessOutput for WMV decoder DMO.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/mf/tests/transform.c:
> DEFINE_GUID(mft_output_sample_incomplete,0xffffff,0xffff,0xffff,0xff,0xff,0xff,0xff,0xff,0xff,0xff,0xff);
>
> +struct buffer_impl
> +{
> +    IMediaBuffer IMediaBuffer_iface;
> +    LONG refcount;
> +    BYTE *data;
> +    DWORD length;
> +    DWORD max_length;
> +};

```suggestion:-7+0
struct media_buffer
{
    IMediaBuffer IMediaBuffer_iface;
    LONG refcount;
    DWORD length;
    DWORD max_length;
    BYTE data[];
};
```

With struct name and method prefixes changed everywhere else, and you can then use a single alloc / free (with allocation size being `offsetof(struct media_buffer, data[size])`).

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2247#note_24982