- if ((DWORD)(Pool) >= 4) {
ERR("(%p) Called with invalid pool type\n", This);
return D3DERR_INVALIDCALL;
- }
You could use if(Pool != D3DPOOL_DEFAULT && Pool != MANAGED ...) instead of comparing to the arbitary value "4". The compiler should figure that out and generate a >= 4 for better efficiency, but even if not it doesn't hurt because Create* isn't performance critical.