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