status-im / nimbus-eth1

Nimbus: an Ethereum Execution Client for Resource-Restricted Devices
https://status-im.github.io/nimbus-eth1/
Apache License 2.0
566 stars 110 forks source link

incomplete EIP161 implementation, account's contract code and storage trie are not removed #440

Open jangko opened 4 years ago

jangko commented 4 years ago

according to EIP161, empty accounts shall be removed after transaction execution.

current implementation: empty accounts are removed, but the contract code and storage trie are not removed.

but removing those contract code and storage trie is tricky. two or more accounts might be share the same contract code or share the same storage trie.

related issue: status-im/nim-eth#9.

mjfh commented 4 months ago

Regarding contract code:

Regarding storage data: