[MIRROR] Removes overly optimistic warn on byond version (#701) (#1770)

* Removes overly optimistic warn on byond version (#81185) ## About The Pull Request I assumed this would be fixed because I assumed it was a bug. My report goes unresponded to. * Removes overly optimistic warn on byond version --------- Co-authored-by: NovaBot <[email protected]> Co-authored-by: LemonInTheDark <[email protected]>
Fluffy-Frontier · Feb 1, 2024 · d513c01 · d513c01
1 parent 09f80d4
commit d513c01
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/code/controllers/globals.dm b/code/controllers/globals.dm
@@ -15,13 +15,12 @@ GLOBAL_REAL(GLOB, /datum/controller/global_vars)
 
 	var/datum/controller/exclude_these = new
 	// I know this is dumb but the nested vars list hangs a ref to the datum. This fixes that
+	// I have an issue report open, lummox has not responded. It might be a FeaTuRE
+	// Sooo we gotta be dumb
 	var/list/controller_vars = exclude_these.vars.Copy()
 	controller_vars["vars"] = null
 	gvars_datum_in_built_vars = controller_vars + list(NAMEOF(src, gvars_datum_protected_varlist), NAMEOF(src, gvars_datum_in_built_vars), NAMEOF(src, gvars_datum_init_order))
 
-#if MIN_COMPILER_VERSION >= 515 && MIN_COMPILER_BUILD > 1620
-	#warn datum.vars hanging a ref should now be fixed, there should be no reason to remove the vars list from our controller's vars list anymore
-#endif
 	QDEL_IN(exclude_these, 0) //signal logging isn't ready
 
 	Initialize()