Skip to content

1.22 refactor #59

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

1.22 refactor #59

wants to merge 2 commits into from

Conversation

@dima-dencep
Copy link
Collaborator

@dima-dencep dima-dencep commented Nov 7, 2025

NOT EARLIER THAN 1.21.11/1.22

What I did:

  • Finally removed AnimationProcessor (I never liked it) and now everything is done in AvatarAnimManager instead.
  • The AvatarAnimationController class has been created in minecraft-independent module, which allows stuff such as geyser ext and others to not copy logic from the Minecraft module.
  • Also removed AvatarAnimationData because it is unnecessary.

@ZigyTheBird
Copy link
Owner

ZigyTheBird commented Nov 7, 2025

LGTM
Also despite your disapproval I have been working on reworking MoLang to comply with the Mojang rendering changes, so whenever 1.21.11 comes out we might have enough changes to bump the version to 2.0.0

@dima-dencep
Copy link
Collaborator Author

dima-dencep commented Nov 7, 2025

If it's the Molang rework I'm thinking of, I'll throw it away

Repository owner deleted a comment from ZigyTheBird Nov 7, 2025
Base automatically changed from compilerupdate to main November 7, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dima-dencep @ZigyTheBird