Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

audit: reduce cognitive overhead #45

Open
rainbreak opened this issue Aug 26, 2016 · 2 comments
Open

audit: reduce cognitive overhead #45

rainbreak opened this issue Aug 26, 2016 · 2 comments
Labels

Comments

@rainbreak
Copy link
Contributor

auction.sol is complicated. Contributors need to remember a fair bit of constraints. One possible change is to recombine the assertions and the business logic to make it clearer what the limits are on variables as they are used.

@rainbreak rainbreak changed the title reduce cognitive overhead audit: reduce cognitive overhead Aug 26, 2016
@rainbreak
Copy link
Contributor Author

Defer for the short term as the problem with this is in having new contributors write secure code, which is harder with the current cognitive load.

@rainbreak
Copy link
Contributor Author

I'm not sure if it makes it any simpler, but one way of doing this is to combine all of the contracts into a single large one. I've done this as part of trying to debug dapple 0.8 and it comes to ~800 lines.

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

No branches or pull requests

1 participant