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

The CellID can have leading zeros #4

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

Conversation

evenly-epic-mule
Copy link

e.g. spawn_point_id 0x "4xxxxxxxxxx" = 0b0100 would be shifted by 21
while spawn_point_id 0x "8xxxxxxxxxx" = 0b1000 would be shifted by 20

cellid_to_location shouldn't do the shifting as it doesn't know the level

additionally monocle (currently only used in the WIP branch) should be updated so that spawnid_to_* will shift the spawnid << 20 before calling cellid_to_*

e.g.  spawn_point_id 0x "4xxxxxxxxxx" = 0b0100 would be shifted by 21
while spawn_point_id 0x "8xxxxxxxxxx" = 0b1000 would be shifted by 20

cellid_to_location shouldn't do the shifting as it doesn't know the level
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.

None yet

1 participant