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

Update naming for a few JRB ship object variables #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jedestep
Copy link

A few naming updates for JRB ship related object fields.

oShipPart3UnkF4 -> oShipPart3PitchAdjust: this variable is used to adjust the pitch of the ship to control the rocking behaviour.
oShipPart3UnkF8 -> oShipPart3RollAdjust: this variable is never written to, but it is used to calculate the ship's roll. Seems like the ship was originally intended to roll back and forth in addition to pitching but this idea was scrapped partway through.
oJrbSlidingBoxUnkF4 -> oJrbSlidingBoxShipCenterReference: this field is a reference to the inSunkenShip object, which the sliding box uses as a parent to keep track of its current position on the ship deck. inSunkenShip is an invisible object inside the floating ship that tracks the ship's overall position, since the tangible ship itself is composed of multiple different sections.

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.

1 participant