Not sure if this leaks or not...

[Acorn511]

I used a leak checker and it said it was fine but I'm not sure if this creates a leak. I bolded the part in question. Also, if it does, how would I fix it? Thanks.


Namerefresher
Events
Time - Every 1.00 seconds of game time
Conditions
Actions
Set Player_Count = (Number of players in (All players matching ((((Matching player) controller) Equal to User) and (((Matching player) slot status) Equal to Is playing))))
For each (Integer A) from 1 to (1 + Player_Count), do (Actions)
Loop - Actions
Multiboard - Set the display style for (Last created multiboard) item in column 1, row 1 to Show text and Hide icons
Multiboard - Set the display style for (Last created multiboard) item in column 1, row (Integer A) to Show text and Show icons
Multiboard - Set the display style for (Last created multiboard) item in column 2, row (Integer A) to Show text and Hide icons
Multiboard - Set the display style for (Last created multiboard) item in column 3, row (Integer A) to Show text and Hide icons
Multiboard - Set the display style for (Last created multiboard) item in column 4, row (Integer A) to Show text and Hide icons
Multiboard - Set the width for (Last created multiboard) item in column 1, row (Integer A) to 2.00% of the total screen width
Multiboard - Set the width for (Last created multiboard) item in column 2, row (Integer A) to 8.00% of the total screen width
Multiboard - Set the width for (Last created multiboard) item in column 3, row (Integer A) to 3.50% of the total screen width
Multiboard - Set the width for (Last created multiboard) item in column 4, row (Integer A) to 3.50% of the total screen width
Set List = 2
Player Group - Pick every player in (All players matching (((((Matching player) slot status) Equal to Is playing) or (((Matching player) slot status) Equal to Has left the game)) and (((Matching player) controller) Equal to User))) and do (Actions)
Loop - Actions
Set Multiboard_Spots[(Player number of (Picked player))] = List
Multiboard - Set the icon for (Last created multiboard) item in column 1, row List to ReplaceableTextures\CommandButtons\BTNPeon.blp
Multiboard - Set the text for (Last created multiboard) item in column 2, row List to (Player_Colors[(Player number of (Picked player))] + ((Name of (Picked player)) + |r))
Set List = (List + 1)
Multiboard - Show (Last created multiboard)

 

[Maker]

I think it leaks.

• Set Temp_Player_Group = (All players matching (...
• Player Group - Pick every player in Temp_Player_Group and do (Actions)
  • Loop - Actions
    • ...
• Custom script: call DestroyForce(udg_Temp_Player_Group)

 

[Acorn511]

Thanks =D

Hopefully the lag on my map will cut down a bit with this.

 

[Maker]

I don't think you need to set those widths and display styles every second. Only when then number of rows changes, and not necessarily even then.