dlls/wined3d/arb_program_shader.c activate "dead" code
Gerald Pfeifer
gerald at pfeifer.com
Thu Nov 22 15:42:51 CST 2007
This, in turn, looks like a real bugfix. The else-branch of
this if-statement never would have triggered since both reg and
This->rel_offset are unsigned types, and so is their difference.
Gerald
ChangeLog:
Rewrite condition in vshader_program_add_param() to actually distinguish
between two cases.
Index: dlls/wined3d/arb_program_shader.c
===================================================================
RCS file: /home/wine/wine/dlls/wined3d/arb_program_shader.c,v
retrieving revision 1.92
diff -u -3 -p -r1.92 arb_program_shader.c
--- dlls/wined3d/arb_program_shader.c 21 Nov 2007 13:48:57 -0000 1.92
+++ dlls/wined3d/arb_program_shader.c 22 Nov 2007 21:39:27 -0000
@@ -468,7 +468,7 @@ static void vshader_program_add_param(SH
break;
case WINED3DSPR_CONST:
if(param & WINED3DSHADER_ADDRMODE_RELATIVE) {
- if(reg - This->rel_offset >= 0) {
+ if(reg >= This->rel_offset) {
sprintf(tmpReg, "C[A0.x + %u]", reg - This->rel_offset);
} else {
sprintf(tmpReg, "C[A0.x - %u]", -reg + This->rel_offset);
More information about the wine-patches
mailing list