py-evm icon indicating copy to clipboard operation
py-evm copied to clipboard

Journal / JournalDB could use a refactor

Open lithp opened this issue 6 years ago • 0 comments

This came during in a review of a PR which touched this file. The current implementation duplicates some logic which could probably be pulled out into a helper function. The duplication caused a bug in JournalDB.diff().

A different abstraction also might make this code simpler to understand.

lithp avatar Apr 18 '19 21:04 lithp