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
Advanced Riding fix #228
Advanced Riding fix #228
Conversation
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.
Good progress for pally QoL.
@@ -356,7 +356,9 @@ int TBeing::doGet(const char *a) | |||
if (fight()) | |||
return FALSE; // don't fall through | |||
} | |||
if (dynamic_cast<TBeing *>(riding)) { | |||
if ((riding && dynamic_cast<TBeing *>(riding)) |
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.
Why all these parentheses? We need none of them.
|
||
if (getPosition() > POSITION_STANDING && | ||
!(getPosition() == POSITION_MOUNTED && | ||
doesKnowSkill(SKILL_ADVANCED_RIDING) && (getSkillValue(SKILL_ADVANCED_RIDING) >= 50))) { |
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.
The doesKnowSkill looks redundant with getSkillValue. Is it possible to have skill value and still not know it?
act("$n sits down in a position of penance.", TRUE, this, 0, 0, TO_ROOM); | ||
setPosition(POSITION_RESTING); | ||
} else { | ||
sendTo("You begin to chant.\n\r"); |
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.
Can you move your mount while doing penance?
gunna open a new PR on this from a different branch. |
Making some QOL improvements for mounted deikhans.