Re: [PATCH v2 0/9] MR6157: d3dx9: More miscellaneous fixes.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/volume.c:
>  if (filter == D3DX_DEFAULT)
>      filter = D3DX_FILTER_TRIANGLE | D3DX_FILTER_DITHER;
>
> - if (FAILED(hr = d3dx9_validate_filter(filter)))
> -
>    return hr;
> -

Sooo, when I proposed this in the previous review I was figuring that `D3DX_DEFAULT` handling could go in the helper as well, although I didn't exactly consider that `D3DXFilterTexture()` has its own different interpretation of `D3DX_DEFAULT`.

What do you think about introducing yet another helper to be used in everything-but-`D3DXFilterTexture()` which does D3DX_DEFAULT handling and also calls `d3dx9_validate_filter()`? E.g. something like `HRESULT d3dx9_handle_load_filter(D3DX_FILTER *filter);`