Trigger Checking Request

[WanderingPeanut]

My map is nearly ready to release but I have one major problem: the map will cause a Fatal Error.
I've been through the obvious things and fixed all the leaks I can see, but the map ill still cause a Fatal error within an hour or so. I've heard people say that if your map uses too much computing power or memory or something like that it will fatal error even with no bad triggers, and my map is pretty CPU intensive at times.

Anyway I was hoping someone could take a look through it, give it a test and see if you know what's wrong, you'll be credited in my map if you have any help!

You can download it here.

 

[Maker]

Leaks:

Test mode

• Unit - Create 1 Tester for (Triggering player) at (Target of current camera view) facing Default building facing degrees

Spawn North, East, South, Centre are full of location leaks.

RandomizationNHostiles (all three triggers) are full of location leaks.

Do ZomCapture like this, and use Triggerin Unit instead of Dying Unit, it's faster.

Trigger

• ZomCapture
  • Events
    • Unit - A unit Dies
  • Conditions
    • ((Killing unit) is Undead) Equal to True
    • ((Dying unit) is A structure) Equal to True
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Dying unit)) Equal to City (Barracks)
      • Then - Actions
        • Unit - Replace (Dying unit) with a Zombie Infested Ruins using The new unit's max life and mana
        • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Dying unit)) Equal to City
          • Then - Actions
            • Unit - Replace (Dying unit) with a Zombie Infested Ruins using The new unit's max life and mana
            • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
          • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Unit-type of (Dying unit)) Equal to Farm
              • Then - Actions
                • Unit - Replace (Dying unit) with a Zombie Infested Farmland using The new unit's max life and mana
                • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
              • Else - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Unit-type of (Dying unit)) Equal to Farm (Neutral Hostile)
                  • Then - Actions
                    • Unit - Replace (Dying unit) with a Zombie Infested Farmland using The new unit's max life and mana
                    • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
                  • Else - Actions
                    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                      • If - Conditions
                        • (Unit-type of (Dying unit)) Equal to Intense Farm
                      • Then - Actions
                        • Unit - Replace (Dying unit) with a Zombie Infested Farmland using The new unit's max life and mana
                        • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
                      • Else - Actions
                        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                          • If - Conditions
                            • (Unit-type of (Dying unit)) Equal to Mechanised Farm
                          • Then - Actions
                            • Unit - Replace (Dying unit) with a Zombie Infested Farmland using The new unit's max life and mana
                            • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
                          • Else - Actions
                            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                              • If - Conditions
                                • (Unit-type of (Dying unit)) Equal to Sparse Farm
                              • Then - Actions
                                • Unit - Replace (Dying unit) with a Zombie Infested Farmland using The new unit's max life and mana
                                • Unit - Change ownership of (Last replaced unit) to (Owner of (Killing unit)) and Change color
                              • Else - Actions

Nuke is full of leaks, unit group and location.

Radioactive Decay 1 and 2 leak.

Farm animals leaks.

Research select leaks unit group.

ZResilience leaks unit group.

BuildFac leaks location.

Oil and Ind leak unit groups.

 

[WanderingPeanut]

Thanks for the help.
By the way I thought Last Created Unit didn't leak (I thought it was global of some kind as there is only ever one Last Created Unit)... so that explains most of the leakage for RandomizationNHostiles, although I now feel like an epicly nooby mapmaker.

 

[Maker]

Last created unit doesn't leak, that's correct.

These do leak:

• Unit - Order (Last created unit) to build a Farm (Neutral Hostile) at (Position of (Last created unit))

^Position of...

• If ((Random integer number between 1 and 5) Equal to 1) then do (Unit - Create 1 Basic Soldier for Neutral Hostile at (Position of (Last created unit)) facing...

^Position of...

Use
loc = position of (last created unit)
and then clear the leak.

 

[ThisPOT]

Hey I have a simple fix for that.

Use the trig: CENTER region at position of unit, and then use this region over and over.

 

[Maker]

That is not a good solution. You still have to set the position of unit to a variable. And if you have done that, you could just use the variable instead of the region. If you use the region, you'd have to set the center of the region to a variable. I don't really see how that would do any good.