From 0f3a11f5bdcd15e21d877978ee02c05c99eb39eb Mon Sep 17 00:00:00 2001 From: Mothblocks <35135081+Mothblocks@users.noreply.github.com> Date: Sat, 17 Dec 2022 02:37:50 -0800 Subject: [PATCH] Replace moveToNullspace in decal Destroy() to loc = null -- Saves 0.11s of init time (#71706) This calls Exited(), Entered(), etc etc etc etc, and it is not necessary for this. Was 119ms, now 9. --- code/game/objects/effects/decals/decal.dm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/code/game/objects/effects/decals/decal.dm b/code/game/objects/effects/decals/decal.dm index 2c4e3fc164d..40e622b5225 100644 --- a/code/game/objects/effects/decals/decal.dm +++ b/code/game/objects/effects/decals/decal.dm @@ -62,5 +62,8 @@ var/turf/T = loc T.RemoveElement(/datum/element/decal, icon, icon_state, dir, null, null, alpha, color, null, FALSE, null) #endif - moveToNullspace() + // Intentionally used over moveToNullspace(), which calls doMove(), which fires + // off an enormous amount of procs, signals, etc, that this temporary effect object + // never needs or affects. + loc = null return QDEL_HINT_QUEUE