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

generate missing methods in SerializedOfflinePlayer at runtime #2777

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

Conversation

wuangg
Copy link

@wuangg wuangg commented Feb 1, 2024

Use ByteBuddy to generate missing methods in SerializedOfflinePlayer from its interface at runtime to prevent compile errors from future API changes.

Fixes #2758.

@wuangg wuangg force-pushed the bytebuddy-serialized-offline-player branch from 785867b to cd45943 Compare February 1, 2024 20:22
@wuangg wuangg changed the title runtime generate missing methods in SerializedOfflinePlayer generate missing methods in SerializedOfflinePlayer at runtime Feb 1, 2024
@dmulloy2
Copy link
Owner

dmulloy2 commented Feb 6, 2024

Would be interested in the performance implications of this. Could you run some tests?

@andrewkm
Copy link

Thank you for this guys! <3

@dmulloy2
Copy link
Owner

dmulloy2 commented Jun 5, 2024

Would be interested in the performance implications of this. Could you run some tests?

@wuangg bump on this :)

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.

Compilation Failure: SerializedOfflinePlayer Missing getLocation Override
3 participants