Re: IWineD3DDeviceImpl_SetRenderState

[Jochen]

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