While you're at it, could you remove all string types and just make them types?

So for example:

>ammo_type = "/obj/item/ammo_casing/c38"

to

>ammo_type = /obj/item/ammo_casing/c38

The major difference is the last one will notify you if a path type is invalid, which is very useful.