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
Port loyaltycascade from Ruby to Lua #542
Conversation
if cache.next_slot > status_slot then | ||
cache.next_slot = status_slot | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what if there were status slots taken above status_slot
? does everything need to be moved down by one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly I'm unsure, this is a 1:1 from the ruby script and testing is difficult without an actual cascade happening. Let me know what your gut says here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say add a comment noting the possibility so we have a place to look if things go wonky. I can do this post merge.
oh, also is there a loyaltycascade.rb file to delete? |
9038fa9
to
3753e47
Compare
if cache.next_slot > status_slot then | ||
cache.next_slot = status_slot | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say add a comment noting the possibility so we have a place to look if things go wonky. I can do this post merge.
Needs to be tested before merging, would be great to find a v50.05 compatible save suffering from a loyalty cascade.
Once merged should be ticked off in DFHack/dfhack#2081