Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow insect regeneration #45241

Merged
merged 2 commits into from
Nov 5, 2020
Merged

Conversation

Venera3
Copy link
Member

@Venera3 Venera3 commented Nov 4, 2020

Summary

SUMMARY: None

Purpose of change

Insects currently can't naturally regenerate HP, since that code only checks for the material types "flesh" or "veggy". This PR adds "iflesh" as an accepted material too.

Alternatives

Changing the material of insects to flesh, and adding the JSON flag "Arthropod blood" one-by-one, keeping this all JSON.

Testing

Compiled, spawned a cow, a triffid and a centipede in closed rooms of a LMOE Shelter. Whacked them with a stick till they got down to red health, teleported away and waited a day. After teleporting back they all were back to full health.

Additional context

The regen rate seems a bit overtuned, with one day allowing a triffid queen to get back to full health, but changing that is definitely a bigger balance consideration than what I wanted to start with.
This is my first time successfully messing with C++, so tell me if I'm touching something I'm not supposed to and I can do the messy JSON fix instead.

@mlangsdorf mlangsdorf added <Enhancement / Feature> New features, or enhancements on existing Monsters Monsters both friendly and unfriendly. labels Nov 4, 2020
@kevingranade kevingranade merged commit 20d1078 into CleverRaven:master Nov 5, 2020
@Venera3 Venera3 deleted the Insect-regen branch November 5, 2020 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants