vgstation-coders / vgstation13

Butts
GNU General Public License v3.0
260 stars 542 forks source link

Silicons have a hit sound and some other fixes and improvements #36750

Open boy2mantwicethefam opened 4 days ago

boy2mantwicethefam commented 4 days ago

Fixes #36752 Attacking silicons now properly plays hitsounds. Cleaned up item_attack a little. The slime item attack code is now its own code in the slime code (and has been slightly tweaked so that it isn't so long, with some minimal gameplay changes). Moved the mob attack code into attacked_by, which has been turned into a living proc instead of just a carbon proc. Silicons make use of their own variant of this proc in order to play hitsounds because hitsounds were only played in the carbon proc. Also fixed a bug where critical damage did not apply to slimes for the purpose of how strong the weapon has to be in order to get rid of them. Slime code has been refactored a little; removed some sleep() and spawn() instances where SStun would wear off afterwards and instead turned SStun into a variable that ticks down every tick. The timings are relatively similar (mostly with Life() timings) and the difference is small enough that nobody will notice. It also helps that repeatedly stunning a slime no longer has a big chance of immediately wearing off because a previous timing was wearing off. Slimes should no longer be included in the global list of things ghost can hear, just like simple_animals. Fixed emotes so that slimes can actually do most of their emotes. Previously it checked if the slime was a monkey. This means the slimes should now actually do more emotes over time instead of just "swaying dizzily". Fixed the "mobs shouldn't play their emotes when no one is near" mechanic even more. It no longer plays emotes even when the type is overridden. Added a client check before trying to send a to_chat to a mob for a failed emote. The pronoun proc will now respect neuter/plural genders by removing "%s" from any emote that has it, and will also work for every living mob instead of just humans. Robots now have a new surrender text.

:cl:

west3436 commented 3 days ago

Unatomic as all hell but a nice change - please try to keep these more atomic in the future for ease of review and bug isolation.

PrimeDSS13 commented 2 hours ago

good changes but unatomic