Fix bytes as fixed type if no ssz tags available

ferranbt / fastssz

Fast Ethereum2.0 SSZ encoder/decoder

MIT License

73 stars 44 forks source link

Closed ferranbt closed 4 months ago

ferranbt commented 4 months ago

Closes #153

ferranbt commented 4 months ago

@itsdevbear the PR changes the encoding generated in the #152 test case as well. The new change looks good to me. Can you verify as well?

itsdevbear commented 4 months ago

LGTM, that Divide 2 stuff is just the chunkify math?

ferranbt commented 4 months ago

LGTM, that Divide 2 stuff is just the chunkify math?

No chunk math there, it makes the int division and checks some corner cases (overflow, out of bounds).