Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce settleLocalVariables struct to alleviate the lack of stack space #1200

Open
pirapira opened this issue Aug 24, 2019 · 0 comments
Open
Labels
efforts: a few days hack P2 demanded by somebody

Comments

@pirapira
Copy link
Contributor

For #1123, TokenNetwork.settleChannel needs more active variables in a scope. For this, we need more local variables than the Solidity compiler allows.

In order to bypass this problem, we can introduce a struct called settleLocalVariables that contains all local variables that would be used. This issue just keeps track of introducing this struct.

@pirapira pirapira added P2 demanded by somebody efforts: a few days hack labels Aug 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
efforts: a few days hack P2 demanded by somebody
Projects
None yet
Development

No branches or pull requests

1 participant