Re: [PATCH 0/2] MR2963: mf/tests: Test input and ouput type for WMA decoder DMO.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/mf/tests/transform.c:
> -
>        info->nChannels, expected->nChannels);
> - ok_(__FILE__, line)(info->nSamplesPerSec == expected->nSamplesPerSec,
> -
>        "Got unexpected nSamplesPerSec %lu, expected %lu.\n",
> -
>        info->nSamplesPerSec, expected->nSamplesPerSec);
> - ok_(__FILE__, line)(info->nAvgBytesPerSec == expected->nAvgBytesPerSec,
> -
>        "Got unexpected nAvgBytesPerSec %lu, expected %lu.\n",
> -
>        info->nAvgBytesPerSec, expected->nAvgBytesPerSec);
> - ok_(__FILE__, line)(info->nBlockAlign == expected->nBlockAlign,
> -
>        "Got unexpected nBlockAlign %u, expected %u.\n",
> -
>        info->nBlockAlign, expected->nBlockAlign);
> - ok_(__FILE__, line)(info->wBitsPerSample == expected->wBitsPerSample,
> -
>        "Got unexpected wBitsPerSample %u, expected %u.\n",
> -
>        info->wBitsPerSample, expected->wBitsPerSample);
> - ok_(__FILE__, line)(info->cbSize == expected->cbSize,
> -
>        "Got unexpected cbSize %u, expected %u.\n",
> -
>        info->cbSize, expected->cbSize);

I don't mind if you want to keep it like this, but FWIW I like check_member macro to save typing the field names and the test messages.