[PATCH 3/6] d3dx9: implement fxlc constants (expressions) in effect.

Paul Gofman gofmanp at gmail.com
Wed Mar 9 07:38:29 CST 2016


Signed-off-by: Paul Gofman <gofmanp at gmail.com>
---
 dlls/d3dx9_36/d3dx9_36_private.h |   5 +
 dlls/d3dx9_36/effect.c           |  66 +++++---
 dlls/d3dx9_36/preshader.c        | 336 +++++++++++++++++++++++++++++++++++++++
 dlls/d3dx9_36/tests/effect.c     |   4 +-
 4 files changed, 383 insertions(+), 28 deletions(-)

diff --git a/dlls/d3dx9_36/d3dx9_36_private.h b/dlls/d3dx9_36/d3dx9_36_private.h
index f3da79d..141781b 100644
--- a/dlls/d3dx9_36/d3dx9_36_private.h
+++ b/dlls/d3dx9_36/d3dx9_36_private.h
@@ -145,5 +145,10 @@ struct d3dx_parameter *get_parameter_by_name(struct d3dx9_base_effect *base,
 HRESULT d3dx_create_preshader(struct d3dx9_base_effect *base_effect, void *byte_code,
         unsigned int byte_code_size, D3DXPARAMETER_TYPE type, struct d3dx_preshader **pres) DECLSPEC_HIDDEN;
 void d3dx_free_preshader(struct d3dx_preshader *pres) DECLSPEC_HIDDEN;
+HRESULT d3dx_pres_evaluate_parameter(struct d3dx_preshader *pres,
+        struct d3dx_parameter *param, void **param_value) DECLSPEC_HIDDEN;
+HRESULT d3dx_pres_set_shader_constants(struct IDirect3DDevice9 *device, struct d3dx_preshader *pres) DECLSPEC_HIDDEN;
+HRESULT d3dx_pres_get_shader_parameters(struct d3dx_preshader *pres, struct d3dx_parameter ***param,
+        unsigned int *nparam, D3DXCONSTANT_DESC **cdesc) DECLSPEC_HIDDEN;
 
 #endif /* __WINE_D3DX9_36_PRIVATE_H */
diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c
index 32241f0..1e468d0 100644
--- a/dlls/d3dx9_36/effect.c
+++ b/dlls/d3dx9_36/effect.c
@@ -2501,13 +2501,19 @@ static HRESULT d3dx9_base_effect_set_array_range(struct d3dx9_base_effect *base,
 static HRESULT d3dx9_get_param_value_ptr(struct ID3DXEffectImpl *effect, struct d3dx_pass *pass,
         struct d3dx_state *state, void **param_value, struct d3dx_parameter **out_param)
 {
-    struct d3dx_parameter *param;
-    param = state->parameter.referenced_param ? state->parameter.referenced_param : &state->parameter;
+    struct d3dx_parameter *param = &state->parameter;
+
+    *param_value = param->data;
+    *out_param = param;
 
     switch (state->type)
     {
         case ST_CONSTANT:
+            *param_value = param->data;
+            *out_param = param;
+            return D3D_OK;
         case ST_PARAMETER:
+            param = param->referenced_param;
             *param_value = param->data;
             *out_param = param;
             return D3D_OK;
@@ -2515,8 +2521,16 @@ static HRESULT d3dx9_get_param_value_ptr(struct ID3DXEffectImpl *effect, struct
             FIXME("Array selector.\n");
             break;
         case ST_FXLC:
-            FIXME("FXLC not supported yet.\n");
-            break;
+            if (param->pres)
+            {
+                *out_param = param;
+                return d3dx_pres_evaluate_parameter(param->pres, param, param_value);
+            }
+            else
+            {
+                FIXME("No preshader for FXLC parameter.\n");
+                break;
+            }
     }
     *param_value = NULL;
     *out_param = NULL;
@@ -2532,19 +2546,19 @@ static void d3dx9_set_light_parameter(enum LIGHT_TYPE op, D3DLIGHT9 *light, void
     }
     light_tbl[] =
     {
-       {FIELD_OFFSET(D3DLIGHT9, Type),         "LC_TYPE"},
-       {FIELD_OFFSET(D3DLIGHT9, Diffuse),      "LT_DIFFUSE"},
-       {FIELD_OFFSET(D3DLIGHT9, Specular),     "LT_SPECULAR"},
-       {FIELD_OFFSET(D3DLIGHT9, Ambient),      "LT_AMBIENT"},
-       {FIELD_OFFSET(D3DLIGHT9, Position),     "LT_POSITION"},
-       {FIELD_OFFSET(D3DLIGHT9, Direction),    "LT_DIRECTION"},
-       {FIELD_OFFSET(D3DLIGHT9, Range),        "LT_RANGE"},
-       {FIELD_OFFSET(D3DLIGHT9, Falloff),      "LT_FALLOFF"},
-       {FIELD_OFFSET(D3DLIGHT9, Attenuation0), "LT_ATTENUATION0"},
-       {FIELD_OFFSET(D3DLIGHT9, Attenuation1), "LT_ATTENUATION1"},
-       {FIELD_OFFSET(D3DLIGHT9, Attenuation2), "LT_ATTENUATION2"},
-       {FIELD_OFFSET(D3DLIGHT9, Theta),        "LT_THETA"},
-       {FIELD_OFFSET(D3DLIGHT9, Phi),          "LT_PHI"}
+        {FIELD_OFFSET(D3DLIGHT9, Type),         "LC_TYPE"},
+        {FIELD_OFFSET(D3DLIGHT9, Diffuse),      "LT_DIFFUSE"},
+        {FIELD_OFFSET(D3DLIGHT9, Specular),     "LT_SPECULAR"},
+        {FIELD_OFFSET(D3DLIGHT9, Ambient),      "LT_AMBIENT"},
+        {FIELD_OFFSET(D3DLIGHT9, Position),     "LT_POSITION"},
+        {FIELD_OFFSET(D3DLIGHT9, Direction),    "LT_DIRECTION"},
+        {FIELD_OFFSET(D3DLIGHT9, Range),        "LT_RANGE"},
+        {FIELD_OFFSET(D3DLIGHT9, Falloff),      "LT_FALLOFF"},
+        {FIELD_OFFSET(D3DLIGHT9, Attenuation0), "LT_ATTENUATION0"},
+        {FIELD_OFFSET(D3DLIGHT9, Attenuation1), "LT_ATTENUATION1"},
+        {FIELD_OFFSET(D3DLIGHT9, Attenuation2), "LT_ATTENUATION2"},
+        {FIELD_OFFSET(D3DLIGHT9, Theta),        "LT_THETA"},
+        {FIELD_OFFSET(D3DLIGHT9, Phi),          "LT_PHI"}
     };
     switch (op)
     {
@@ -2559,7 +2573,7 @@ static void d3dx9_set_light_parameter(enum LIGHT_TYPE op, D3DLIGHT9 *light, void
             D3DCOLORVALUE c = *(D3DCOLORVALUE *)value;
 
             TRACE("%s (%f %f %f %f).\n", light_tbl[op].name, c.r, c.g, c.b, c.a);
-            *(D3DCOLORVALUE *)((char *)light + light_tbl[op].offset) = c;
+            *(D3DCOLORVALUE *)((BYTE *)light + light_tbl[op].offset) = c;
             break;
         }
         case LT_POSITION:
@@ -2568,7 +2582,7 @@ static void d3dx9_set_light_parameter(enum LIGHT_TYPE op, D3DLIGHT9 *light, void
             D3DVECTOR v = *(D3DVECTOR *)value;
 
             TRACE("%s (%f %f %f).\n", light_tbl[op].name, v.x, v.y, v.z);
-            *(D3DVECTOR *)((char *)light + light_tbl[op].offset) = v;
+            *(D3DVECTOR *)((BYTE *)light + light_tbl[op].offset) = v;
             break;
         }
         case LT_RANGE:
@@ -2581,7 +2595,7 @@ static void d3dx9_set_light_parameter(enum LIGHT_TYPE op, D3DLIGHT9 *light, void
         {
             float v = *(float *)value;
             TRACE("%s %f.\n", light_tbl[op].name, v);
-            *(float *)((char *)light + light_tbl[op].offset) = v;
+            *(float *)((BYTE *)light + light_tbl[op].offset) = v;
             break;
         }
         default:
@@ -2599,11 +2613,11 @@ static void d3dx9_set_material_parameter(enum MATERIAL_TYPE op, D3DMATERIAL9 *ma
     }
     material_tbl[] =
     {
-       {FIELD_OFFSET(D3DMATERIAL9, Diffuse),  "MT_DIFFUSE"},
-       {FIELD_OFFSET(D3DMATERIAL9, Ambient),  "MT_AMBIENT"},
-       {FIELD_OFFSET(D3DMATERIAL9, Specular), "MT_SPECULAR"},
-       {FIELD_OFFSET(D3DMATERIAL9, Emissive), "MT_EMISSIVE"},
-       {FIELD_OFFSET(D3DMATERIAL9, Power),    "MT_POWER"}
+        {FIELD_OFFSET(D3DMATERIAL9, Diffuse),  "MT_DIFFUSE"},
+        {FIELD_OFFSET(D3DMATERIAL9, Ambient),  "MT_AMBIENT"},
+        {FIELD_OFFSET(D3DMATERIAL9, Specular), "MT_SPECULAR"},
+        {FIELD_OFFSET(D3DMATERIAL9, Emissive), "MT_EMISSIVE"},
+        {FIELD_OFFSET(D3DMATERIAL9, Power),    "MT_POWER"}
     };
 
     switch (op)
@@ -2624,7 +2638,7 @@ static void d3dx9_set_material_parameter(enum MATERIAL_TYPE op, D3DMATERIAL9 *ma
             D3DCOLORVALUE c = *(D3DCOLORVALUE *)value;
 
             TRACE("%s, value (%f %f %f %f).\n", material_tbl[op].name, c.r, c.g, c.b, c.a);
-            *(D3DCOLORVALUE *)((char *)material + material_tbl[op].offset) = c;
+            *(D3DCOLORVALUE *)((BYTE *)material + material_tbl[op].offset) = c;
             break;
         }
         default:
diff --git a/dlls/d3dx9_36/preshader.c b/dlls/d3dx9_36/preshader.c
index 536514a..ee4db90 100644
--- a/dlls/d3dx9_36/preshader.c
+++ b/dlls/d3dx9_36/preshader.c
@@ -199,6 +199,12 @@ static void val_set_reg(struct d3dx_fxlc *fxlc, unsigned int table, unsigned int
             1 << (reg_idx % PRES_VS_BITS_PER_WORD);
 }
 
+static DWORD is_val_set_reg(struct d3dx_fxlc *fxlc, unsigned int table, unsigned int reg_idx)
+{
+    return fxlc->table_value_set[table][reg_idx / PRES_VS_BITS_PER_WORD] &
+            1 << (reg_idx % PRES_VS_BITS_PER_WORD);
+}
+
 static void dump_shader_bytecode(void *data, unsigned int size)
 {
     DWORD *words = (DWORD *)data;
@@ -669,3 +675,333 @@ void d3dx_free_preshader(struct d3dx_preshader *pres)
     HeapFree(GetProcessHeap(), 0, pres);
 }
 
+static HRESULT set_constants_param(struct d3dx_fxlc *fxlc, struct d3dx_const_tab *const_tab,
+        D3DXHANDLE hc, struct d3dx_parameter *param)
+{
+    ID3DXConstantTable *ctab = const_tab->ctab;
+    D3DXCONSTANT_DESC desc;
+    unsigned int c_cnt, p_cnt, i, j, n, table, start_offset;
+    unsigned int nminor, nmajor, major_stride, inp_ind;
+    BOOL transpose;
+
+    ID3DXConstantTable_GetConstantDesc(ctab, hc, &desc, &c_cnt);
+    while (param->member_count && param->element_count)
+    {
+        if (param->element_count > 1)
+        {
+            FIXME("Unexpected param having both elements and members.\n");
+            return D3DERR_INVALIDCALL;
+        }
+        param = &param->members[0];
+    }
+
+    if (desc.Elements > 1)
+        c_cnt = desc.Elements;
+    else
+        c_cnt = desc.StructMembers;
+    p_cnt = MAX(param->element_count, param->member_count);
+    if (c_cnt != p_cnt)
+    {
+        FIXME("Number of elements or struct members differs between parameter and constant.\n");
+        return D3DERR_INVALIDCALL;
+    }
+    if (c_cnt)
+    {
+        HRESULT hr, ret;
+        D3DXHANDLE hcelem;
+
+        ret = D3D_OK;
+        for (i = 0; i < c_cnt; i++)
+        {
+            if (desc.StructMembers)
+                hcelem = ID3DXConstantTable_GetConstant(ctab, hc, i);
+            else
+                hcelem = ID3DXConstantTable_GetConstantElement(ctab, hc, i);
+            if (hcelem == NULL)
+            {
+                FIXME("Could not get constant.\n");
+                hr = D3DERR_INVALIDCALL;
+            }
+            else
+                hr = set_constants_param(fxlc, const_tab, hcelem, &param->members[i]);
+            if (FAILED(hr))
+                ret = hr;
+        }
+        return ret;
+    }
+
+    transpose = (desc.Class == D3DXPC_MATRIX_COLUMNS && param->class == D3DXPC_MATRIX_ROWS) ||
+            (param->class == D3DXPC_MATRIX_COLUMNS && desc.Class == D3DXPC_MATRIX_ROWS);
+    if (desc.Class == D3DXPC_MATRIX_COLUMNS)
+    {
+        nmajor = param->columns;
+        nminor = param->rows;
+    }
+    else
+    {
+        nmajor = param->rows;
+        nminor = param->columns;
+    }
+    TRACE("%s rows %u, par columns %u, par class %u, par flags %u, par bytes %u, c rows %u,"\
+            "c columns %u, c class %u, c bytes %u, transpose %u.\n",
+            desc.Name, param->rows, param->columns, param->class, param->flags, param->bytes,
+            desc.Rows, desc.Columns, desc.Class, desc.Bytes, transpose);
+    table = const_tab->regset2table[desc.RegisterSet];
+    start_offset = desc.RegisterIndex * table_info[table].reg_value_cnt;
+    major_stride = MAX(nminor, table_info[table].reg_value_cnt);
+    n = MIN(nmajor * major_stride,
+            desc.RegisterCount * table_info[table].reg_value_cnt + major_stride - 1) / major_stride;
+    for (i = 0; i < n; i++)
+    {
+        for (j = 0; j < nminor; j++)
+        {
+            BYTE *out;
+            DWORD *in;
+            unsigned int offset;
+
+            offset = start_offset + i * major_stride + j;
+            if (offset / table_info[table].reg_value_cnt >= fxlc->table_sizes[table])
+            {
+                if (table_info[table].reg_value_cnt != 1)
+                    FIXME("Output offset exceeds table size, name %s, component %u.\n", desc.Name, i);
+                break;
+            }
+            if (transpose)
+                inp_ind = i + j * nmajor;
+            else
+                inp_ind = i * nminor + j;
+            if (inp_ind * sizeof(DWORD) >= param->bytes)
+            {
+                WARN("Parameter data is too short, name %s, component %u.\n", desc.Name, i);
+                break ;
+            }
+
+            in = (DWORD *)param->data + inp_ind;
+            out = (BYTE *)fxlc->tables[table] + offset * table_info[table].value_size;
+            /* TODO: store data transfer / convert operation instead of performing an operation
+                from here, to move this to parsing stage */
+            switch (table_info[table].type)
+            {
+                case PRES_VT_FLOAT: set_number(out, D3DXPT_FLOAT, in, param->type); break;
+                case PRES_VT_INT: set_number(out, D3DXPT_INT, in, param->type); break;
+                case PRES_VT_BOOL: set_number(out, D3DXPT_BOOL, in, param->type); break;
+                default:
+                    FIXME("Unexpected type %#x.\n", table_info[table].type);
+                    break;
+            }
+            val_set_reg(fxlc, table, offset / table_info[table].reg_value_cnt);
+        }
+    }
+
+    return D3D_OK;
+}
+
+static HRESULT set_constants(struct d3dx_fxlc *fxlc, struct d3dx_const_tab *const_tab)
+{
+    unsigned int i;
+    HRESULT hr, ret;
+    D3DXHANDLE hc;
+
+    ret = D3D_OK;
+    for (i = 0; i < const_tab->ninputs; i++)
+    {
+        if (const_tab->inputs_param[i] == NULL || const_tab->inputs_param[i]->class == D3DXPC_OBJECT)
+            continue;
+        hc = ID3DXConstantTable_GetConstant(const_tab->ctab, NULL, i);
+        if (hc == NULL)
+        {
+            FIXME("Could not get constant.\n");
+            hr = D3DERR_INVALIDCALL;
+        }
+        else
+            hr = set_constants_param(fxlc, const_tab, hc, const_tab->inputs_param[i]);
+        if (FAILED(hr))
+            ret = hr;
+    }
+    return ret;
+}
+
+static float exec_get_arg(struct d3dx_fxlc *fxlc, struct d3dx_pres_ins *ins,
+        struct d3dx_pres_operand *opr, unsigned int comp)
+{
+    BYTE *p;
+
+    p = (BYTE *)fxlc->tables[opr->table] + table_info[opr->table].value_size * (opr->offset + comp);
+    if (!is_val_set_reg(fxlc, opr->table, (opr->offset + comp) / table_info[opr->table].reg_value_cnt))
+    {
+        WARN("Using uninitialized input ");
+        dump_arg(fxlc, opr, comp);
+        TRACE(".\n");
+        dump_ins(fxlc, ins);
+    }
+    switch (table_info[opr->table].type)
+    {
+        case PRES_VT_FLOAT : return *(float *)p;
+        case PRES_VT_DOUBLE: return *(double *)p;
+        case PRES_VT_INT   : return *(int *)p;
+        case PRES_VT_BOOL  : return *(BOOL *)p ? 1.0f : -0.0f;
+        default: return NAN;
+    }
+}
+
+static void exec_set_arg(struct d3dx_fxlc *fxlc, struct d3dx_pres_operand *opr, unsigned int comp, float res)
+{
+    BYTE *p;
+
+    p = (BYTE *)fxlc->tables[opr->table] + table_info[opr->table].value_size * (opr->offset + comp);
+    switch (table_info[opr->table].type)
+    {
+        case PRES_VT_FLOAT : *(float *)p = res; break;
+        case PRES_VT_DOUBLE: *(double *)p = res; break;
+        case PRES_VT_INT   : *(int *)p = roundf(res); break;
+        case PRES_VT_BOOL  : *(BOOL *)p = !!res; break;
+    }
+    val_set_reg(fxlc, opr->table, (opr->offset + comp) / table_info[opr->table].reg_value_cnt);
+}
+
+#define ARGS_ARRAY_SIZE 8
+static HRESULT execute_preshader(struct d3dx_fxlc *fxlc)
+{
+    unsigned int i, j, k;
+    float args[ARGS_ARRAY_SIZE];
+    float res;
+
+    for (i = 0; i < fxlc->nins; i++)
+    {
+        struct d3dx_pres_ins *ins;
+        struct op_info *oi;
+
+        ins = &fxlc->ins[i];
+        oi = &pres_op_info[ins->op];
+        if (oi->func_all_comps)
+        {
+            if (ins->ninp_args * ins->ncomps > ARGS_ARRAY_SIZE)
+            {
+                FIXME("Too much arguments (%u) for one instruction.\n", ins->ninp_args * ins->ncomps);
+                return E_FAIL;
+            }
+            for (k = 0; k < ins->ninp_args; k++)
+                for (j = 0; j < ins->ncomps; j++)
+                    args[k * ins->ncomps + j] = exec_get_arg(fxlc, ins, &ins->inputs[k],
+                            ins->scalar_op && !k ? 0 : j);
+            res = oi->func(args, ins->ncomps);
+
+            /* only 'dot' instruction currently falls here */
+            exec_set_arg(fxlc, &ins->output, 0, res);
+        }
+        else
+        {
+            for (j = 0; j < ins->ncomps; j++)
+            {
+                for (k = 0; k < ins->ninp_args; k++)
+                    args[k] = exec_get_arg(fxlc, ins, &ins->inputs[k], ins->scalar_op && !k ? 0 : j);
+                res = oi->func(args, ins->ncomps);
+                exec_set_arg(fxlc, &ins->output, j, res);
+            }
+        }
+    }
+    return D3D_OK;
+}
+
+HRESULT d3dx_pres_evaluate_parameter(struct d3dx_preshader *pres, struct d3dx_parameter *param, void **param_value)
+{
+    HRESULT hr;
+    unsigned int i;
+    unsigned int elements, elements_param, elements_table;
+    float *oc;
+
+    TRACE("pres %p, param %p, param_value %p.\n", pres, param, param_value);
+
+    hr = set_constants(&pres->fxlc, &pres->fxlc.inputs);
+    if (FAILED(hr))
+        return hr;
+
+    hr = execute_preshader(&pres->fxlc);
+    if (FAILED(hr))
+        return hr;
+
+    elements_table = table_info[PRES_TAB_OCONST].reg_value_cnt * pres->fxlc.table_sizes[PRES_TAB_OCONST];
+    elements_param = param->bytes / sizeof(DWORD);
+    elements = MIN(elements_table, elements_param);
+    *param_value = param->data;
+    oc = (float *)pres->fxlc.tables[PRES_TAB_OCONST];
+    for (i = 0; i < elements; i++)
+        set_number((DWORD *)(*param_value) + i, param->type, oc + i, D3DXPT_FLOAT);
+    return D3D_OK;
+}
+
+HRESULT d3dx_pres_set_shader_constants(struct IDirect3DDevice9 *device, struct d3dx_preshader *pres)
+{
+    HRESULT hr, res;
+    struct d3dx_fxlc *fxlc = &pres->fxlc;
+
+    TRACE("device %p, pres %p, param_type %u.\n", device, pres, pres->param_type);
+
+    if (pres->pres_type != PRES_TYPE_SHADER)
+    {
+        FIXME("Called for non-shader type preshader.\n");
+        return D3DERR_INVALIDCALL;
+    }
+
+    if (FAILED(hr = set_constants(fxlc, &fxlc->inputs)))
+        return hr;
+    if (FAILED(hr = execute_preshader(fxlc)))
+        return hr;
+    if (FAILED(hr = set_constants(fxlc, &pres->shader_inputs)))
+        return hr;
+
+    #define SET_TABLE(table, func_suff, type) \
+        {\
+            unsigned int is, n;\
+            is = 0;\
+            while (is < fxlc->table_sizes[table])\
+            {\
+                n = 0;\
+                while (is < fxlc->table_sizes[table] && !is_val_set_reg(fxlc, table, is))\
+                    is++;\
+                while (is + n < fxlc->table_sizes[table] && is_val_set_reg(fxlc, table, is + n))\
+                    n++;\
+                if (!n) continue ;\
+                TRACE("setting %u consts at %u.\n", n, is);\
+                hr = IDirect3DDevice9_##func_suff(device, is,\
+                        (const type *)fxlc->tables[table] + is * table_info[table].reg_value_cnt, n);\
+                if (FAILED(hr))\
+                {\
+                    ERR(#func_suff" failed, hr %#x.\n", hr);\
+                    res = hr;\
+                }\
+                is += n;\
+            }\
+        }
+    res = D3D_OK;
+    if (pres->param_type == D3DXPT_VERTEXSHADER)
+    {
+        SET_TABLE(PRES_TAB_OCONST,  SetVertexShaderConstantF, float);
+        SET_TABLE(PRES_TAB_OICONST, SetVertexShaderConstantI, int);
+        SET_TABLE(PRES_TAB_OBCONST, SetVertexShaderConstantB, BOOL);
+    }
+    else if (pres->param_type == D3DXPT_PIXELSHADER)
+    {
+        SET_TABLE(PRES_TAB_OCONST,  SetPixelShaderConstantF, float);
+        SET_TABLE(PRES_TAB_OICONST, SetPixelShaderConstantI, int);
+        SET_TABLE(PRES_TAB_OBCONST, SetPixelShaderConstantB, BOOL);
+    }
+    else
+    {
+        FIXME("Unexpected parameter type %u.\n", pres->param_type);
+        return D3DERR_INVALIDCALL;
+    }
+    #undef SET_TABLE
+
+    return res;
+}
+
+HRESULT d3dx_pres_get_shader_parameters(struct d3dx_preshader *pres, struct d3dx_parameter ***param,
+        unsigned int *nparam, D3DXCONSTANT_DESC **cdesc)
+{
+    *nparam = pres->shader_inputs.ninputs;
+    *param = pres->shader_inputs.inputs_param;
+    *cdesc = pres->shader_inputs.inputs;
+    return D3D_OK;
+}
+
diff --git a/dlls/d3dx9_36/tests/effect.c b/dlls/d3dx9_36/tests/effect.c
index 8c5b5bb..f7336c9 100644
--- a/dlls/d3dx9_36/tests/effect.c
+++ b/dlls/d3dx9_36/tests/effect.c
@@ -2915,7 +2915,7 @@ static void test_effect_states(IDirect3DDevice9 *device)
     ok(!memcmp(mat.m, test_mat.m, sizeof(mat)), "World matrix does not match.\n");
 
     hr = effect->lpVtbl->BeginPass(effect, 0);
-    todo_wine ok(hr == D3D_OK, "Got result %x, expected 0 (D3D_OK).\n", hr);
+    ok(hr == D3D_OK, "Got result %x, expected 0 (D3D_OK).\n", hr);
 
     hr = IDirect3DDevice9_GetTransform(device, D3DTS_WORLDMATRIX(1), &mat);
     ok(hr == D3D_OK, "Got result %x, expected 0 (D3D_OK).\n", hr);
@@ -2923,7 +2923,7 @@ static void test_effect_states(IDirect3DDevice9 *device)
 
     hr = IDirect3DDevice9_GetTransform(device, D3DTS_VIEW, &mat);
     ok(hr == D3D_OK, "Got result %x, expected 0 (D3D_OK).\n", hr);
-    todo_wine ok(!memcmp(mat.m, test_mat_camera.m, sizeof(mat)), "View matrix does not match.\n");
+    ok(!memcmp(mat.m, test_mat_camera.m, sizeof(mat)), "View matrix does not match.\n");
 
     hr = IDirect3DDevice9_GetRenderState(device, D3DRS_BLENDOP, &value);
     ok(hr == D3D_OK, "Got result %x, expected 0 (D3D_OK).\n", hr);
-- 
2.5.0




More information about the wine-patches mailing list