IWineD3DDeviceImpl_SetRenderState

Jochen blackdrag at gmx.org
Sun Jul 27 16:51:08 CDT 2008


Chris Ahrendt schrieb:
[...]
> hmmmm...  I do know without the patch EQ2 doesn't run and with it it 
> then gets  into the start of the game.. so it might be another one of 
> the if's... I can add the fog to the break with no if's...let me try 
> that. Right now I am running just the git 1.1.2+ tree from this morning 
> plus the fix in 12929 which does this :

your patch contained this (shortened):

> +    switch(State)
> +    {
[...]
> +     case WINED3DRS_FOGCOLOR:
> +          /* Valid Values are between 0 and FFFF (4 bytes alpha, red, green, and blue)) */
> +          if ((Value >= 0) || 
> +              (Value <= 0xFFFF)) break;
> +          return WINED3DERR_INVALIDCALL;
> +     break;
[...]
> +
> +     case WINED3DRS_BLENDOPALPHA:
> +          WINED3DRS_DEPTHBIAS:
[...]
> +          WINED3DRS_FOGSTART:
> +          WINED3DRS_FOGEND:
> +          WINED3DRS_FOGDENSITY:
> +          WINED3DRS_CLIPPLANEENABLE:
> +          WINED3DRS_POINTSIZE:
> +          WINED3DRS_POINTSIZE_MIN:
> +          WINED3DRS_POINTSCALE_A:
> +          WINED3DRS_POINTSCALE_B:
> +          WINED3DRS_POINTSCALE_C:
> +          break;
>  
> +     Default:
> +        return WINED3DERR_INVALIDCALL;
> +    } 

so unless you changed

>> +          if ((Value >= 0) || 
>> +              (Value <= 0xFFFF)) break;
>> +          return WINED3DERR_INVALIDCALL;

into:

>> +          break;

or added WINED3DRS_FOGCOLOR to the case block that doesn't do any checks 
(before the default), it wouldn't make me wonder if it still works

just as an idea

bye jochen




More information about the wine-devel mailing list