[PATCH 1/5] d3dx9: Turn D3DXCreateEffectEx FIXME into a TRACE, improve it a bit.

[Henri Verbeet]

On 3 March 2014 16:28, Matteo Bruni <mbruni at codeweavers.com> wrote:
> ---
>  dlls/d3dx9_36/effect.c | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c
> index a4325af..0ec17c1 100644
> --- a/dlls/d3dx9_36/effect.c
> +++ b/dlls/d3dx9_36/effect.c
> @@ -5294,8 +5294,10 @@ HRESULT WINAPI D3DXCreateEffectEx(struct IDirect3DDevice9 *device, const void *s
>      struct ID3DXEffectImpl *object;
>      HRESULT hr;
>
> -    FIXME("(%p, %p, %u, %p, %p, %p, %#x, %p, %p, %p): semi-stub\n", device, srcdata, srcdatalen, defines, include,
> -        skip_constants, flags, pool, effect, compilation_errors);
> +    TRACE("device %p, srcdata %p, srcdatalen %u, defines %p, include %p,\n",
> +            device, srcdata, srcdatalen, defines, include);
> +    TRACE("skip_constants %p, flags %#x, pool %p, effect %p, compilation_errors %p.\n",
> +            skip_constants, flags, pool, effect, compilation_errors);
>
Mostly just FWIW, I think it's generally nicer to keep that as a single TRACE.