Leak Check Help!

I am creating a maze map, and using TFT editor. It's very hard to make a region on every side of every path, including all the diagonal pathing I have, which you can't make diagonal regions. I tried using "Maze Region Writer" to write a trigger for me. That did not work, and all my regions were messed up, from the diagonals and all. So I made a trigger that I hoped would work. It is as follows:









I used the program "Leak Checker," and it tells me to fix it by doing this:


(Suggested fix) Set L = (Position of (Picked unit))) Equal to Dalaran - Black Marble) and (((Picked unit) is A Hero) Equal to True)
(Suggested fix) Custom Script: Call RemoveLocation(udg_L)

Where do I get (Position of (Picked unit) without using Terrain type at position of picked unit? And what is "Set L?"

L is a Point variable, every time you pick a unit you create a point and that can cause massive laggs. (I know it, cause i make a escape on myself)
You set L = Position of Picked Unit and when it's checked you destroy the point again, so the leak is minimal.

Firstly, please use trigger tags so it's easier to read your trigger.
Second, I think I can help you out. Give me a minute so I can check to be sure I'm correct, and I'll give you directions.
Dammit Fireeye, you beat me ^_^.

Here's instructions anyways, in case you may need them.
1) make a Point(location) variable in the variable editor, name it L.
2) Use custom script, and type "local location udg_L"
3) After the condition that checks whether your unit is on the black marble, then put in the following action part the action:
Set Variable L = (position of (picked unit))).
4) continue your trigger, but after the Kill unit action, put in custom script saying set local L = null and you should be good to go!

P.S. thx for fixing it to trigger tags

custom script is an action. I gave you instructions in my above post.

woops you must have just done the instructions

Yea, It took a bit to type them up for you. Hope they make sense. If not, let me know and I'll make the trigger up for you.
Will I get repped for this?...^_^

Thank you so much for helping me! There is one more problem, it says after doing this, that it is expecting a Variable name on the "set local L= null" thing. What do I do to fix that?

Lol of course you will be repped for this, you've been a great help.

Also, running it again in Leak Checker, it's still asking me to do:

(Suggested fix) Set L = (Position of (Picked unit))) Equal to Dalaran - Black Marble) and (((Picked unit) is A Hero) Equal to True) which I can't do in actions.

Sorry for all the trouble. :(

no problem, I more than happy to help. Looks like I made a mistake. Replace the bad script with this one, and It should work.
set udg_L = null
I made a mistake, making locals in GUI is confusing.
Once you fix that, I think the leak checker will change too. If not, I don't know what else leaks... But I don't think it's too major. Also, you may want to set your periodic timer to a slightly smaller amount, maybe .10.
I enjoy helping people whith problems, just as long as I know how to actually fix it...lol. That's why I can rarely help, becuase either I don't know how to fix it, or someone beat me to it.

Same like me, btw. when you don't ask for a trigger solution i won't give it.
Well, i prefer this way. It works too.

Yea, both work fine in this case. I prefer to do it after, so I can see exactly where it comes from.
And although I try to help people, many times people beat me to it, so even though I know the answer, I don't get to share...
Also, I really dislike indirectly making locals in GUI (i mean like helping somebody) becuase then I make mistakes, like i did a minute before. Ah well, prblem solved. I'm happy I helped.

lmao ty guys so much.. but guess what? Leakage:


Current trigger I made by your help:

Terrain leaks? I don't think it does. I'm not quite sure what it's asking, but I'll look into it. Give me a minute or two, and I'll try to help.
EDIT: woah lol it does. There is a variable called Terrain Type, but I don't think you can set 2 locals variables in GUI. I honestly don't think it will lag that much. I don't know how to fix this one... The only way I can think of is by switching to JASS. You know what, how about I do it for you here, since I need to work on my JASS. I'll use your trigger and turn it into a non-lag trigger. :D I'll post it when I have a chance.

i know what it wants, but it's not a leak (i think), he want you to use the variable L instead of Position of (picked Unit)

Leakless trigger:

Oh that works... Lol didn't know thats what he wanted completely forgot about setting picked unit to L. Well, that savs me from making a huge JASS trigger *hooray!*