kills 4 object definitions that are inside of proc arguments (#61152)

thanks to putnam for reporting this in coderbus
/obj/machinery/smartfridge/drying_rack/load(/obj/item/dried_object)
looks pretty harmless right? youd think that this would make a proc argument called dried_object of type /obj/item but no, this defines a new /obj/item class definition called dried_object with zero changes from /obj/item

you can use this to do weird things, like /obj/machinery/smartfridge/drying_rack/load(/obj/item/var/dried_object = 3)
creates a new var on /obj/item called dried_object and is very cursed
gets rid of 4 of these abominations, they dont seem to have caused any bugs except for maybe behavior that relies on typesof(/some/path)
This commit is contained in:
Kylerace
2021-09-05 11:54:40 +01:00
committed by GitHub
parent 4f00c9dfa1
commit 87f6ca1944
4 changed files with 4 additions and 4 deletions
+1 -1
View File
@@ -95,7 +95,7 @@ Simple datum which is instanced once per type and is used for every object of sa
source.mat_update_desc(src)
///This proc is called when a material updates an object's description
/atom/proc/mat_update_desc(/datum/material/mat)
/atom/proc/mat_update_desc(datum/material/mat)
return
///This proc is called when the material is added to an object specifically.
+1 -1
View File
@@ -300,7 +300,7 @@
SEND_SIGNAL(quirk_holder, COMSIG_CLEAR_MOOD_EVENT, "nyctophobia")
/// Called when the quirk holder moves. Updates the quirk holder's mood.
/datum/quirk/nyctophobia/proc/on_holder_moved(/mob/living/source, atom/old_loc, dir, forced)
/datum/quirk/nyctophobia/proc/on_holder_moved(mob/living/source, atom/old_loc, dir, forced)
SIGNAL_HANDLER
if(quirk_holder.stat == DEAD)
@@ -308,7 +308,7 @@
if(!powered())
toggle_drying(TRUE)
/obj/machinery/smartfridge/drying_rack/load(/obj/item/dried_object) //For updating the filled overlay
/obj/machinery/smartfridge/drying_rack/load(obj/item/dried_object) //For updating the filled overlay
. = ..()
update_appearance()
+1 -1
View File
@@ -591,7 +591,7 @@ GLOBAL_LIST_INIT(gun_saw_types, typecacheof(list(
update_appearance()
return TRUE
/obj/item/gun/ballistic/proc/guncleaning(mob/user, /obj/item/A)
/obj/item/gun/ballistic/proc/guncleaning(mob/user, obj/item/A)
if(misfire_probability == 0)
to_chat(user, span_notice("[src] seems to be already clean of fouling."))
return