Moderator
M
Moderator
Bribe:
Unless needed, your variables should be normal variables instead of arrays. Key thing, if you can type the value of an array as an actual integer (0, 1, 2), it can be split into normal variables. Normal variables execute faster and are overall much better.
SoEL_Index is a fine examples of this... you never reference anything other than slots 1, 2, 3, which means this should be split into three variables.
The bj_wantDestroyGroup custom script is also a lot faster than setting a group and then destroying it.
Overall a good spell, I can't spot any leaks. But this is a huge efficiency boost that could be given to the system and I recommend implementing such changes before I approve this.
Unless needed, your variables should be normal variables instead of arrays. Key thing, if you can type the value of an array as an actual integer (0, 1, 2), it can be split into normal variables. Normal variables execute faster and are overall much better.
SoEL_Index is a fine examples of this... you never reference anything other than slots 1, 2, 3, which means this should be split into three variables.
The bj_wantDestroyGroup custom script is also a lot faster than setting a group and then destroying it.
Overall a good spell, I can't spot any leaks. But this is a huge efficiency boost that could be given to the system and I recommend implementing such changes before I approve this.