Post reply

Warning: this topic has not been posted in for at least 120 days.
Unless you're sure you want to reply, please consider starting a new topic.
Name:
Email:
Subject:
Message icon:
Verification:
Type the letters shown in the picture
Listen to the letters / Request another image

Type the letters shown in the picture:
What color is grass?:
What is the seventh word in this sentence?:
What is five minus two (use the full word)?:

shortcuts: hit alt+s to submit/post or alt+p to preview


Topic Summary

Posted by: Lollorian
« on: November 29, 2011, 08:41:52 PM »

<insert epic rage face>
Such a simple solution for something I spent over 8 hrs trying to debug! :o

Thanks again for your help mate! :D
Posted by: the bigg
« on: November 29, 2011, 03:49:28 PM »

That's because you're never resetting the amulet variable.

This code:
Code: [Select]
SET "%variable%" = 5doesn't work. You need to use
Code: [Select]
SET "variable" = 5or
Code: [Select]
SET variable = 5
Posted by: Lollorian
« on: November 29, 2011, 05:12:27 AM »

WeiDU seems to skip random PATCH_IFs even if all the conditions match when doing COPY_EXISTING_REGEXP ??? This one to be exact (there are others as well but I debugged just this :D)
Code: [Select]
            PATCH_IF (("%type%" =  1) AND ("%amulet%" = 0) AND ("%ignore%" = 0) AND (("%flags%" BAND 0b00000100) = 0b00000100) AND ("%index%" > 17)) BEGIN
              PATCH_PRINT ~%SOURCE_FILE%: Equippable items in inventory: %item%.itm! Equipping AMULET~
              WRITE_SHORT ("%slot_off%" + (0x02 * "%index%")) 0xffff
              WRITE_SHORT ("%slot_off%" + (0x02 * 6))         "ref"
              SET "%amulet%" = 1
            END ELSE
I added this line to output the values used in the condition
Code: [Select]
PATCH_PRINT ~%SOURCE_FILE%: = %item% - %type% - %amulet% - %ignore% - %flags% - %index%~right above the condition.

Now when doing a COPY_EXISTING of the individual file, it seems to properly move the amulet
Code: [Select]
Copying and patching 1 file ...

CBELEDI1.CRE: = MINHP1 - 1 - 0 - 0 - 40 - 4

CBELEDI1.CRE: = CLCK20 - 32 - 0 - 0 - 108 - 21

CBELEDI1.CRE: Equippable items in inventory: CLCK20.itm! Equipping CLOAK

CBELEDI1.CRE: = WAND07 - 35 - 0 - 0 - 108 - 22

CBELEDI1.CRE: = AMUL21 - 1 - 0 - 0 - 108 - 23

CBELEDI1.CRE: Equippable items in inventory: AMUL21.itm! Equipping AMULET

CBELEDI1.CRE: = SCRL7J - 11 - 0 - 0 - 108 - 24

CBELEDI1.CRE: = SCRL6N - 11 - 0 - 0 - 108 - 25
Copied [CBELEDI1.CRE] to [override/CBELEDI1.CRE].. but in the regex copy, this happens...
Code: [Select]
CBDRWW23.CRE: = POTN02 - 9 - 1 - 0 - 108 - 27
Copied [CBDRWW23.CRE] to [override/CBDRWW23.CRE]

CBELEDI1.CRE: = MINHP1 - 1 - 0 - 0 - 40 - 4

CBELEDI1.CRE: = CLCK20 - 32 - 0 - 0 - 108 - 21

CBELEDI1.CRE: Equippable items in inventory: CLCK20.itm! Equipping CLOAK

CBELEDI1.CRE: = WAND07 - 35 - 0 - 0 - 108 - 22

CBELEDI1.CRE: = AMUL21 - 1 - 0 - 0 - 108 - 23

CBELEDI1.CRE: = SCRL7J - 11 - 0 - 0 - 108 - 24

CBELEDI1.CRE: = SCRL6N - 11 - 0 - 0 - 108 - 25
Copied [CBELEDI1.CRE] to [override/CBELEDI1.CRE]

CBELEDI2.CRE: = REGHP1 - 10 - 0 - 0 - 40 - 4
Note that the conditions for both cases are exactly identical, but the outcome seems to differ :o
The unmodified errant CRE, the .tp2 and both DEBUGs attached here :) Sorry for the trouble and dragging you into all this :P