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

Bugfix: Items over 255 durability #5773

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kphoenix137
Copy link
Collaborator

Some items with the of Structure power can exceed 255 durability, causing the item to drop to a low durability when regenerated. This clamps the bonus given from the affix so that it doesn't exceed 255.

StephenCWills
StephenCWills previously approved these changes Sep 30, 2023
@AJenbo
Copy link
Member

AJenbo commented Sep 30, 2023

Should it not be clamped to 254?

@StephenCWills
Copy link
Member

Hm, maybe yes, if you assume #5772 won't be merged any time soon. I was kinda hoping it was close to ready, though I can't say I know what we should do about Oil of Permanence.

@kphoenix137
Copy link
Collaborator Author

approved, when merge?

Source/items.cpp Outdated Show resolved Hide resolved
@AJenbo
Copy link
Member

AJenbo commented Sep 15, 2024

approved, when merge?

See previous comments, or the one I just made.

@StephenCWills StephenCWills dismissed their stale review September 15, 2024 12:54

Comments that need to be addressed

Co-authored-by: Anders Jenbo <anders@jenbo.dk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants