Re: [PATCH v4 0/2] MR979: d3d9: Implement d3dx_effect_SetRawValue

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/effect.c:
> +                    test_set_raw_data_values[x].param_name, hr,
> +                    test_set_raw_data_values[x].expected_result );
> +            effect->lpVtbl->GetValue( effect,
> +                    effect->lpVtbl->GetParameterByName( effect, NULL, test_set_raw_data_values[x].param_name ),
> +                    result_buf,
> +                    test_set_raw_data_values[x].expected_data_len );
> +            switch (test_set_raw_data_values[x].param_type)
> +            {
> +                case D3DXPT_BOOL:
> +                case D3DXPT_INT:
> +                case D3DXPT_FLOAT:
> +                    single_size = (test_set_raw_data_values[x].param_type == D3DXPT_BOOL) ? sizeof( BOOL ) :
> +                            (test_set_raw_data_values[x].param_type == D3DXPT_INT) ? sizeof( DWORD ) :
> +                            sizeof( FLOAT );
> +
> +                    if (test_set_raw_data_values[x].expected_data_len > single_size)
> What's the point of this check?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/979#note_11100