[MIRROR] Document why moths eating clothes is like that [MDB IGNORE] (#19319)

* Document why moths eating clothes is like that (#73382)

People have tried to change this a few times and I want to make sure
it's written in stone that I'm not stupid, just forward thinking

* Document why moths eating clothes is like that

---------

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>
This commit is contained in:
SkyratBot
2023-02-13 10:52:46 -08:00
committed by GitHub
co-authored by Mothblocks
parent 5e8b016ea9
commit d3d3a88cf9
+9 -2
View File
@@ -40,7 +40,15 @@
/// How many zones (body parts, not precise) we have disabled so far, for naming purposes
var/zones_disabled
/// A lazily initiated "food" version of the clothing for moths
/// A lazily initiated "food" version of the clothing for moths.
// This intentionally does not use the edible component, for a few reasons.
// 1. Effectively everything that wants something edible, from now and into the future,
// does not want to receive clothing, simply because moths *can* eat it.
// 2. Creating this component for all clothing has a non-negligible impact on init times and memory.
// 3. Creating the component contextually to solve #2 will make #1 much more confusing,
// and frankly not be a better solution than what we are doing now.
// The first issue could be solved if "edible" checks were more granular,
// such that you never actually cared about checking if something is *edible*.
var/obj/item/food/clothing/moth_snack
/obj/item/clothing/Initialize(mapload)
@@ -66,7 +74,6 @@
if(M.putItemFromInventoryInHandIfPossible(src, H.held_index))
add_fingerprint(usr)
//This code is cursed, moths are cursed, and someday I will destroy it. but today is not that day.
/obj/item/food/clothing
name = "temporary moth clothing snack item"
desc = "If you're reading this it means I messed up. This is related to moths eating clothes and I didn't know a better way to do it than making a new food object. <--- stinky idiot wrote this"