-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Completely replace fastnbt
with our own NBT crate
#420
Comments
fastnbt
with our own NBT cratefastnbt
with our own NBT crate
Now that I am out of the translation issue, can I take up this one? |
Yes, Please would be very nice |
@asodugf12 How its going ? |
oop, Ive been a bit busy the past few days so I'll start work now. Can I get a briefing on the problems with the current system? |
So Serializing (data->NBT) works fine, no problems. But Deserializing (NBT->data) which is done in |
Is there any specific function that doesnt work, or what is the function output against the desired output. I forgot my charger, so there id a limited time window I could work on things and i just wanna make sure i could be most productive |
Hey, I'm not sure, its just not working, I first tough it had something to do with Arrays but after testing not only Arrays are not working, the whole Deserialization is broken. You don't have any "limited time window", But ofc as sooner as better |
I've searched existing issues and couldn't find a duplicate.
Is your feature request related to a problem? Please describe.
Our own NBT crate currently does work great when serializing values into NBT, But deserialzing seems not to really work
Describe the solution you'd like
Fix our NBT crate and finally remove
fastnbt
. This should also finally allow #283 to be mergedThe text was updated successfully, but these errors were encountered: