Stefan Dösinger wrote:
My performance concern is about the switch statement itself, less about the actual value checking. If you have something like
switch(renderstate) { case WINED3DRS_A: case WINED3DRS_B: doSomething(); break;
default: break;}
The binary code generated by the compiler still checks 239 states against D3DRS_A and D3DRS_B(the only exception is D3DRS_A, which is never compared against D3DRS_B). That is where my worries come from, not the doSomething() itself.
But beyond that, I think checking the value ranges is just wrong, at least we need a test to show Windows does that as well.
Ok I will work on the test case when I get a chance... (I think I can do this in a vmware session if not it is going to have to wait till I get to the house where I have a windows machine).
Ahh ok... I also this morning moved the check to see if the call is just resetting the same value over and over to before the switch statement so that if we are getting the same value it just returns like it did before however it does not set it over again.
Chris