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

By jove, I think I've done it #244

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

Conversation

Draco18s
Copy link
Contributor

@Draco18s Draco18s commented Jul 2, 2023

World -> IWorld was the last piece. The worldgen being threaded meant that by trying to read/write to ServerWorld instead of the pseudo worldgen abstract region, it was locking up, two threads waiting on each other.

Behold, success:

pic

Commands are still inoperable, have no idea if the wireframe rendering code needs its own updates, I didn't even mess with those last time. Not critical to my own build path, though certainly handy.

The literal BS feature/structure registration I did having worked without any issue is a huge load off my conscience as it means I don't need to try and figure out what the hell COG does or how structures work. Did the literal bare bones required for the game to accept things at compile-time, hoping I didn't have to go deeper, worked out.

Now I can start on updating for 1.19, which hopefully is just updating mappings for the most part, though I'm sure a few other oddities abound as well, eg. the world now extends down to y=-64.

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