[PATCH v2 03/10] d3dx9: Merge the d3dx_effect_SetInt() helper
Michael Stefaniuc
mstefani at winehq.org
Sat Mar 9 04:21:26 CST 2019
Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
---
v2: FLOAT ==> float
dlls/d3dx9_36/effect.c | 80 +++++++++++++++++++-----------------------
1 file changed, 36 insertions(+), 44 deletions(-)
diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c
index 7425c7b9a1..d1eaa6a767 100644
--- a/dlls/d3dx9_36/effect.c
+++ b/dlls/d3dx9_36/effect.c
@@ -1345,49 +1345,6 @@ static HRESULT d3dx9_base_effect_get_value(struct d3dx9_base_effect *base,
return D3DERR_INVALIDCALL;
}
-static HRESULT d3dx9_base_effect_set_int(struct d3dx9_base_effect *base, D3DXHANDLE parameter, INT n)
-{
- struct d3dx_parameter *param = get_valid_parameter(base, parameter);
-
- if (param && !param->element_count)
- {
- if (param->rows == 1 && param->columns == 1)
- {
- DWORD value;
-
- set_number(&value, param->type, &n, D3DXPT_INT);
- if (value != *(DWORD *)param->data)
- set_dirty(param);
- *(DWORD *)param->data = value;
- return D3D_OK;
- }
-
- /*
- * Split the value, if parameter is a vector with dimension 3 or 4.
- */
- if (param->type == D3DXPT_FLOAT &&
- ((param->class == D3DXPC_VECTOR && param->columns != 2) ||
- (param->class == D3DXPC_MATRIX_ROWS && param->rows != 2 && param->columns == 1)))
- {
- TRACE("Vector fixup\n");
-
- *(FLOAT *)param->data = ((n & 0xff0000) >> 16) * INT_FLOAT_MULTI_INVERSE;
- ((FLOAT *)param->data)[1] = ((n & 0xff00) >> 8) * INT_FLOAT_MULTI_INVERSE;
- ((FLOAT *)param->data)[2] = (n & 0xff) * INT_FLOAT_MULTI_INVERSE;
- if (param->rows * param->columns > 3)
- {
- ((FLOAT *)param->data)[3] = ((n & 0xff000000) >> 24) * INT_FLOAT_MULTI_INVERSE;
- }
- set_dirty(param);
- return D3D_OK;
- }
- }
-
- WARN("Parameter not found.\n");
-
- return D3DERR_INVALIDCALL;
-}
-
static HRESULT d3dx9_base_effect_set_int_array(struct d3dx9_base_effect *base,
D3DXHANDLE parameter, const INT *n, UINT count)
{
@@ -3351,10 +3308,45 @@ static HRESULT WINAPI d3dx_effect_GetBoolArray(ID3DXEffect *iface, D3DXHANDLE pa
static HRESULT WINAPI d3dx_effect_SetInt(ID3DXEffect *iface, D3DXHANDLE parameter, INT n)
{
struct d3dx_effect *effect = impl_from_ID3DXEffect(iface);
+ struct d3dx_parameter *param = get_valid_parameter(&effect->base_effect, parameter);
TRACE("iface %p, parameter %p, n %d.\n", iface, parameter, n);
- return d3dx9_base_effect_set_int(&effect->base_effect, parameter, n);
+ if (param && !param->element_count)
+ {
+ if (param->rows == 1 && param->columns == 1)
+ {
+ DWORD value;
+
+ set_number(&value, param->type, &n, D3DXPT_INT);
+ if (value != *(DWORD *)param->data)
+ set_dirty(param);
+ *(DWORD *)param->data = value;
+ return D3D_OK;
+ }
+
+ /*
+ * Split the value, if parameter is a vector with dimension 3 or 4.
+ */
+ if (param->type == D3DXPT_FLOAT &&
+ ((param->class == D3DXPC_VECTOR && param->columns != 2) ||
+ (param->class == D3DXPC_MATRIX_ROWS && param->rows != 2 && param->columns == 1)))
+ {
+ TRACE("Vector fixup.\n");
+
+ *(float *)param->data = ((n & 0xff0000) >> 16) * INT_FLOAT_MULTI_INVERSE;
+ ((float *)param->data)[1] = ((n & 0xff00) >> 8) * INT_FLOAT_MULTI_INVERSE;
+ ((float *)param->data)[2] = (n & 0xff) * INT_FLOAT_MULTI_INVERSE;
+ if (param->rows * param->columns > 3)
+ ((float *)param->data)[3] = ((n & 0xff000000) >> 24) * INT_FLOAT_MULTI_INVERSE;
+ set_dirty(param);
+ return D3D_OK;
+ }
+ }
+
+ WARN("Parameter not found.\n");
+
+ return D3DERR_INVALIDCALL;
}
static HRESULT WINAPI d3dx_effect_GetInt(ID3DXEffect *iface, D3DXHANDLE parameter, INT *n)
--
2.20.1
More information about the wine-devel
mailing list