Moderator
M
Moderator
12.12
IcemanBo: For long time as NeedsFix. Rejected.
Bribe
This appears to be leaking a fair amount of locations, but it's hard to tell based on your strange use of storing locations into a variable before storing them into a hashatble.
(Ordered unit) -> (Triggering unit)
(Owner of (Ordered unit)) -> (Triggering player)
(Center of (Entire map)) leaks (Entire map). Just use (Center of (Playable map area))
((Region centered at -> leaks a region
(Owner of (Triggering unit)) -> (Triggering player)
Don't use arrays if you already know their indices. Instead of "Var[2]" use "Var2". This is faster and also less RAM-consuming and file-size heavy. GUI arrays add a bit of text to the "InitGlobals" function and just a few arrays amount to a good 50 lines of code. Avoid them when possible.
IcemanBo: For long time as NeedsFix. Rejected.
Bribe
This appears to be leaking a fair amount of locations, but it's hard to tell based on your strange use of storing locations into a variable before storing them into a hashatble.
(Ordered unit) -> (Triggering unit)
(Owner of (Ordered unit)) -> (Triggering player)
(Center of (Entire map)) leaks (Entire map). Just use (Center of (Playable map area))
((Region centered at -> leaks a region
(Owner of (Triggering unit)) -> (Triggering player)
Don't use arrays if you already know their indices. Instead of "Var[2]" use "Var2". This is faster and also less RAM-consuming and file-size heavy. GUI arrays add a bit of text to the "InitGlobals" function and just a few arrays amount to a good 50 lines of code. Avoid them when possible.