Skip to content

Add: Chunk handling API - #11

Open
Trident35ro wants to merge 4 commits into
eris-webserv:mainfrom
Trident35ro:chunks
Open

Trident35ro wants to merge 4 commits into
eris-webserv:mainfrom
Trident35ro:chunks

Conversation

@Trident35ro

Copy link
Copy Markdown
Contributor

Allows modders to modify chunk generation or chunk generation rules.

Allows modders to modify chunk generation or chunk generation rules.
ChunkControl doesn't use Unity's Start() message; it implements the
game's custom OrderedStart interface instead (Start_0()/Start_1()).
Patching a literal "Start" made Harmony.PatchAll() throw and abort
before patching anything else in the assembly.
The postfix declared pass-through params "X"/"Z" (HostGetChunk's real
params are chunkX/chunkZ, and Harmony binds pass-through params by
name) and typed __result as Chunk when HostGetChunk actually returns
ChunkData. Harmony's IL patcher threw "Parameter X not found" and
aborted PatchAll before patching anything else in the assembly.

ChunkLoadedEvent updated to carry ChunkData instead of Chunk to match,
and the postfix now actually fires it through EventBus instead of
constructing and discarding it.

@eris-webserv eris-webserv left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs to be de-conflicted.

@Trident35ro

Copy link
Copy Markdown
Contributor Author

The conflict has been solved.

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.

2 participants