Skip to content

Conversation

@haykam821
Copy link
Contributor

This pull request introduces some minor refactors to rendering that remove all references to RenderSystem and ShaderProgramKeys, the latter of which is removed in Minecraft snapshot 25w07a. This change can also be backported to other version branches.

@haykam821 haykam821 changed the base branch from 1.21.4 to 1.21.5 February 14, 2025 00:48
@haykam821 haykam821 requested a review from Prospector February 14, 2025 00:48
@sakura-ryoko
Copy link
Contributor

sakura-ryoko commented Feb 14, 2025

FYI, Shader Program Keys still works; but it isn't the same; and they made it harder to reference / use them. I have them working, personally. ( Example: https://github.com/sakura-ryoko/malilib/blob/d030f3a6daf675a7bd606a37f2c5d0dcb68abccf/src/main/java/fi/dy/masa/malilib/render/RenderContext.java#L204 )

But either way, this is still important work. I nearly have PAPI ready also, just waiting on PRE.

@gniftygnome
Copy link
Contributor

Some of this was merged in #861. Do you want to update the rest to merge into 1.21.11?

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.

3 participants