ethstorage / optimism

Optimism is Ethereum, scaled.
https://optimism.io
MIT License
2 stars 3 forks source link

contracts-bedrock: add some view functions to FaultDisputeGameN #66

Closed JustXxx closed 1 month ago

JustXxx commented 2 months ago

To support op-challenger's functionality, this PR slightly modified FaultDisputeGameN.sol:

Regression tests

forge test --mc FaultDisputeGameN_Test forge test --mc FaultDisputeN_1v1_Actors_Test

dajuguan commented 2 months ago

Why attack branch variable's type need to be unify with other variable types?

JustXxx commented 1 month ago

Why attack branch variable's type need to be unify with other variable types?

When calling in Go language, there is no need to handle different data types.