-
Notifications
You must be signed in to change notification settings - Fork 4
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
Replace codegen with bindnode and add plain IPLD schema #39
Conversation
79518ad
to
b333ff6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much cleaner!
@@ -63,11 +69,11 @@ func (n *Node) _bucketSize() int { | |||
// gets coalesced to -1. | |||
// We also prevent overflows and underflows, as we convert int64 to int. | |||
const maxBucketSize = 1 << 20 // 1MiB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, here's a max, but oh boy it's big! we might have to revisit that later, and I don't see an equivalent here for bitWidth which could have worse effects.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Captured #40
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neato
b333ff6
to
fa3d91a
Compare
Define the HAMT schema in a dedicated ipldsch file and replace the codegen types with go structs using IPLD bindnode. Note the IPLD dependency is updated to the current head of main branch to take advantage of the latest unreleased bindnode fixes.
fa3d91a
to
f1a2395
Compare
Define the HAMT schema in a dedicated ipldsch file and replace the
codegen types with go structs using IPLD bindnode.
Note the IPLD dependency is updated to the current head of main branch
to take advantage of the latest unreleased bindnode fixes.