[5/7] wined3d: Remove some redundant checks in the vertexdeclaration() state handler

H. Verbeet hverbeet at gmail.com
Mon Mar 12 17:22:06 CDT 2007


position_transformed won't be set if the stream doesn't contain
position data, so that part of the check is redundant.

Changelog:
  - Remove some redundant checks in the vertexdeclaration() state handler
-------------- next part --------------
---

 dlls/wined3d/state.c |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c
index 5b4a304..f3acb3b 100644
--- a/dlls/wined3d/state.c
+++ b/dlls/wined3d/state.c
@@ -2905,10 +2905,7 @@ static void vertexdeclaration(DWORD state, IWineD3DStateBlockImpl *stateblock, W
 
     handleStreams(stateblock, useVertexShaderFunction, context);
 
-    /* Do I have to use ? TRUE : FALSE ? Or can I rely on 15==15 being equal to TRUE(=1)? */
-    transformed = ((device->strided_streams.u.s.position.lpData != NULL ||
-                    device->strided_streams.u.s.position.VBO != 0) &&
-                    device->strided_streams.u.s.position_transformed) ? TRUE : FALSE;
+    transformed = device->strided_streams.u.s.position_transformed;
     if (transformed) useVertexShaderFunction = FALSE;
 
     if(transformed != context->last_was_rhw && !useVertexShaderFunction) {


More information about the wine-patches mailing list