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

Make Wyrmskin Armor Repairable Again. #15340

Merged
merged 3 commits into from Feb 11, 2016

Conversation

Projects
None yet
2 participants
@Malkeus
Copy link
Contributor

commented Feb 10, 2016

Making it unrepairable wasn't intended, this PR corrects that. I would welcome input on the adjustment to the wyrmskin materials acid_resist to allow the total acid_resist for the armor to remain the same as before.

Are material resistances averaged on dual material items?

Wyrmskin material acid resist is 20 prior to this PR.
Leather material acid resist is 4.

edit: Unrepairable status was noted in the forums: http://smf.cataclysmdda.com/index.php?topic=11013.msg267022#msg267022

@Malkeus

This comment has been minimized.

Copy link
Contributor Author

commented Feb 10, 2016

Ok, I believe I found the relevant code:
item.ccp:line 3039: resist /= mat_types.size();
So I'm going to set wyrmskin material to acid_resist to 36.

Adjust wyrmskin acid_resist
This should ensure that the total acid_resist of Wyrmskin Armor remains the same as before, 20.

@Malkeus Malkeus changed the title [CR] Make Wyrmskin Armor Repairable Again. Make Wyrmskin Armor Repairable Again. Feb 11, 2016

@kevingranade kevingranade merged commit dcfa9d2 into CleverRaven:master Feb 11, 2016

1 check passed

default
Details

@Malkeus Malkeus deleted the Malkeus:Wyrmskin_Fix branch Feb 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.