No subject

> The "instr" parameter to shader_glsl_gen_modifier() is actually a source
> parameter, so comparing against WINED3DSIO_TEXKILL is just silly. 


What was removed was a single if...return check, which according to the information above makes sense. If it didn't make sense, the other Wine/CodeWeavers DirectX developers would have commented on it.

There are various checks to make sure that a patch is correct: reviews of patches sent to the mailing list; Alexandre (the ultimate reviewer and applier of patches) and conformance tests. This system is not perfect, and regressions do happen (the Windows API is very big and complex).