Skip to content
This repository has been archived by the owner on Aug 5, 2019. It is now read-only.

Switch to integer encoding to little-endian #23

Closed
wemeetagain opened this issue Jan 19, 2019 · 3 comments
Closed

Switch to integer encoding to little-endian #23

wemeetagain opened this issue Jan 19, 2019 · 3 comments

Comments

@wemeetagain
Copy link
Member

ethereum/consensus-specs#139 got merged.
Integers should now be encoded in little-endian format.
This includes prefixed lengths and encoded integers.

@GregTheGreek
Copy link
Member

Just saw this, would be happy to take a stab at this unless you do?

@Mikerah
Copy link
Contributor

Mikerah commented Jan 24, 2019

A little food for thought: Would it be valuable to have the option to encode integers in big endian? SSZ has some really nice properties that others might want to leverage in other projects that may or may not be related to Ethereum. They might require integers encoded in big endian. This could be a future feature and will not be a priority within the next few months.

GregTheGreek added a commit that referenced this issue Jan 24, 2019
@GregTheGreek
Copy link
Member

@Mikerah Just saw this. We can easily just cherry-pick the old code and add support for it. Ill make an issue for the backburner

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants