Wisp Posted January 4, 2009 Share Posted January 4, 2009 Running a diff on the v7 tp2 I found this (line 18630 in the unmodified tp2): WRITE_LONG ("%fx_off%" + 0x08 + (0x30 * ("%abil_fx_idx%" + "%index2%" + 1))) 55 // icon: stun SET "fx_delta" = ("%fx_delta%" + 2) The fix is for the Barbarian Rage spell but as you can see, the fix will not correctly update effect indices since the update to fx_delta is commented out. It will not matter to the vanilla Rage spell, since it only has 1 ability (so I'm not suggesting v7 should be updated because of this, so long as it makes it into v8 ). Link to comment
Leomar Posted January 7, 2009 Share Posted January 7, 2009 If you can tell us, what and where we must fix, we can include a fix for that issue in the next Big World Fixpack. Greetings Leomar Link to comment
Salk Posted January 7, 2009 Share Posted January 7, 2009 I think - in my ignorance - that it's just a simple matter of moving SET "fx_delta" = ("%fx_delta%" + 2) in a line under since like it is now, it is commented out. Link to comment
Jarno Mikkola Posted January 7, 2009 Share Posted January 7, 2009 So that's WRITE_LONG ("%fx_off%" + 0x08 + (0x30 * ("%abil_fx_idx%" + "%index2%" + 1))) 55 // icon: stun SET "fx_delta" = ("%fx_delta%" + 2) instead of a WRITE_LONG ("%fx_off%" + 0x08 + (0x30 * ("%abil_fx_idx%" + "%index2%" + 1))) 55 // icon: stun SET "fx_delta" = ("%fx_delta%" + 2) Copy/paste :( Link to comment
DavidW Posted January 7, 2009 Share Posted January 7, 2009 It probably depends how diligent you're feeling, though, Leomar: Wisp's quite right to point this out (& my bad, I think: copy/paste error), but it's only going to cause trouble in some fairly exotic circumstances. Link to comment
Leomar Posted January 10, 2009 Share Posted January 10, 2009 Thanks for the explanations. I've a small issue with v7, too. A very small issue... In the tp2 is a typo by the German translation. LANGUAGE ~Deutsche (by Leonardo Watson)~ ~german~ ~bg2fixpack/english/setup.tra~ Deutsche must be Deutsch (without an e). It would be nice if you can correct this for the next release. Greetings Leomar Link to comment
Nythrun Posted January 15, 2010 Share Posted January 15, 2010 Added by DavidW for version 8. Link to comment
Recommended Posts
Archived
This topic is now archived and is closed to further replies.