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

Use kotlin flatbuffers #484

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

ImUrX
Copy link
Member

@ImUrX ImUrX commented Jan 17, 2023

This comes with many things.

  • Better typings, everything that in our schema is an unsigned will be seen like that instead of just being an int.
  • Null-safety, nullables are specified now, so we can know if we are getting something that can be null or not.
  • Removal of object API aka the .pack() and .unpack() are gonna go
  • Flatbuffer creation has some nice things with vectors because they use primitive arrays that come with the kotlin std (nice, no boxing)
  • I'm working on adding more type-safety on the offsets so we can have even safer table creation now that there is no object API

@ImUrX ImUrX force-pushed the kotlin-flatbuffers branch from 104a50c to f7bc0ab Compare January 17, 2023 06:40
@ButterscotchV ButterscotchV added the Area: Application Protocol Related to communication with apps like the GUI, overlay, games label Jan 18, 2023
@ImUrX ImUrX force-pushed the kotlin-flatbuffers branch from b489590 to 73a7f8f Compare January 24, 2023 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Application Protocol Related to communication with apps like the GUI, overlay, games
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants