search

mana-ethereum / ex_rlp

Elixir implementation of Ethereum's RLP (Recursive Length Prefix) encoding
MIT License
31 stars 9 forks source link

Update common tests #52

Closed ayrat555 closed 1 year ago

ayrat555 commented 1 year ago

This pr simplifies decoding logic. The code has worse performance but better maintainability

ayrat555 commented 1 year ago

@InoMurko what do you think? the main reasoning behind this pr is that in the previous state of the library some new tests were failing. and to fix them, it would take 1k changes

plus it would be much harder to implement - https://github.com/mana-ethereum/ex_rlp/pull/53