schroedingerscode edited a comment on issue #19: SSZ v0.5.0 Related Changes
URL: https://github.com/apache/incubator-tuweni/pull/19#issuecomment-502211404
 
 
   Variable length lists are still included as of now. The proposal I linked in 
my first comment has some running commentary about what changes to expect to 
the spec. If I've read it correctly, all lists will be variable length lists 
for serialization (and thus will include the mixin), but will have a max size 
defined that is used to pad out to when calculating tree hashes.
   
   This isn't finalized yet, and I wouldn't expect to see it make it into the 
SSZ spec for another 4 or so weeks. What's still unknown is, while all lists 
will have a max size for tree hash purposes, if some lists will still be 
serialized as a fixed size (w/o the mixin), or if all will have a length mixin. 
Everything being variable sized for serialization purposes, and bounded size 
for tree hash calculating purposes makes the most sense to me, so that's what 
I'm assuming will happen.
   
   I will look into @cleishm comments shortly and get a little bit better more 
refactored version out sometime this weekend. Thanks for reviewing.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to