Skip to content

Build: Port to 26.2-snapshot-6 - #137

Merged
Johni0702 merged 1 commit into
masterfrom
feature/26.2-snapshot-6
Jun 3, 2026
Merged

Build: Port to 26.2-snapshot-6#137
Johni0702 merged 1 commit into
masterfrom
feature/26.2-snapshot-6

Conversation

@Johni0702

Copy link
Copy Markdown
Collaborator

No description provided.

@Traben-0
Traben-0 self-requested a review June 2, 2026 04:43

@Traben-0 Traben-0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

//$$
//#endif

//#if MC >= 1.17 && MC < 26.2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

question: what does this change accomplish? why can't it remain in the above elseif block?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It's conceptually separate from the above preprocessor chain. I.e. the fact that this is < 26.2 and that the above uses a separate block for 26.2+ are merely coincidental. Note that there's no 26.2+ replacement for this method, while there is for the one above.

The above chain is for converting the int OpenGL constant to the MC type (because our constructor needs that). It changes with 26.2+ because VertexFormat.DrawMode became PrimitiveTopology.

This method is for converting from the MC type to our type (because our fromRenderLayer needs that). It is removed with 26.2+ because fromRenderLayer is removed because RenderType (aka RenderLayer) no longer exists the way it used to.

@Johni0702
Johni0702 merged commit 7ed6905 into master Jun 3, 2026
1 check passed
@Johni0702
Johni0702 deleted the feature/26.2-snapshot-6 branch June 3, 2026 05:01
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