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.

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

Update deku #1

Merged
merged 7 commits into from
Oct 23, 2023
Merged

Update deku #1

merged 7 commits into from
Oct 23, 2023

Conversation

wcampbell0x2a
Copy link
Owner

@wcampbell0x2a wcampbell0x2a commented Aug 4, 2023 •
edited
Loading

No description provided.

@v1gnesh
Copy link

v1gnesh commented Aug 17, 2023

Hey, a pretty dumb question -
The impl_reader work that you've been working on.
Is that what's making Deserialize blazingly fast?

@wcampbell0x2a
Copy link
Owner Author

Hey, a pretty dumb question - The impl_reader work that you've been working on. Is that what's making Deserialize blazingly fast?

Yep! that's the plan

@wcampbell0x2a
Copy link
Owner Author

Hey, a pretty dumb question - The impl_reader work that you've been working on. Is that what's making Deserialize blazingly fast?

I knew those results were too good to be true. Updated my mistake and now we are in between custom and binrw which I think is fair!

@wcampbell0x2a wcampbell0x2a force-pushed the update-deku branch 5 times, most recently from 6870677 to 24851d8 Compare August 28, 2023 01:53
@wcampbell0x2a wcampbell0x2a mentioned this pull request Aug 28, 2023
Merged
8 tasks
@wcampbell0x2a wcampbell0x2a merged commit 035acec into master Oct 23, 2023
@wcampbell0x2a wcampbell0x2a deleted the update-deku branch October 23, 2023 01:42
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.
2 participants
@wcampbell0x2a @v1gnesh