Closed jalextowle closed 1 month ago
Totals | |
---|---|
Change from base Build 9374786147: | 0.0% |
Covered Lines: | 1833 |
Relevant Lines: | 1963 |
Totals | |
---|---|
Change from base Build 9374786147: | 0.0% |
Covered Lines: | 1833 |
Relevant Lines: | 1963 |
Benchmark suite | Current: 5e6774a6f2a7d9c3dea0c2eb18daa11824f144dc | Previous: 5383ec41cfe4c92ea2831cbbd09bbab8ee94527e | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33827 gas |
33827 gas |
0% |
š° |
addLiquidity: avg |
155541 gas |
156428 gas |
-0.5670% |
ā |
addLiquidity: max |
429092 gas |
429092 gas |
0% |
š° |
checkpoint: min |
40292 gas |
40292 gas |
0% |
š° |
checkpoint: avg |
142306 gas |
142280 gas |
0.0183% |
šØ |
checkpoint: max |
253424 gas |
253424 gas |
0% |
š° |
closeLong: min |
31361 gas |
31361 gas |
0% |
š° |
closeLong: avg |
136118 gas |
135514 gas |
0.4457% |
šØ |
closeLong: max |
2625796 gas |
2625796 gas |
0% |
š° |
closeShort: min |
31349 gas |
31349 gas |
0% |
š° |
closeShort: avg |
131995 gas |
131768 gas |
0.1723% |
šØ |
closeShort: max |
263302 gas |
402203 gas |
-34.5350% |
ā |
initialize: min |
31371 gas |
31371 gas |
0% |
š° |
initialize: avg |
330940 gas |
331025 gas |
-0.0257% |
ā |
initialize: max |
396934 gas |
396934 gas |
0% |
š° |
openLong: min |
33370 gas |
33370 gas |
0% |
š° |
openLong: avg |
174129 gas |
173464 gas |
0.3834% |
šØ |
openLong: max |
306958 gas |
306958 gas |
0% |
š° |
openShort: min |
33936 gas |
33936 gas |
0% |
š° |
openShort: avg |
168522 gas |
168593 gas |
-0.0421% |
ā |
openShort: max |
415910 gas |
415910 gas |
0% |
š° |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
š° |
redeemWithdrawalShares: avg |
75021 gas |
75352 gas |
-0.4393% |
ā |
redeemWithdrawalShares: max |
210128 gas |
210128 gas |
0% |
š° |
removeLiquidity: min |
31301 gas |
31301 gas |
0% |
š° |
removeLiquidity: avg |
214675 gas |
209310 gas |
2.5632% |
šØ |
removeLiquidity: max |
403971 gas |
403971 gas |
0% |
š° |
This comment was automatically generated by workflow using github-action-benchmark.
Totals | |
---|---|
Change from base Build 9374786147: | 0.0% |
Covered Lines: | 1833 |
Relevant Lines: | 1963 |
Totals | |
---|---|
Change from base Build 9374786147: | 0.0% |
Covered Lines: | 1833 |
Relevant Lines: | 1963 |
Description
Adds testing for the LP sandwich attack to ensure that the initial LP will not lose money with a properly configured circuit breaker delta.
Review Checklists
Please check each item before approving the pull request. While going through the checklist, it is recommended to leave comments on items that are referenced in the checklist to make sure that they are reviewed. If there are multiple reviewers, copy the checklists into sections titled
## [Reviewer Name]
. If the PR doesn't touch Solidity, the corresponding checklist can be removed.[[Reviewer Name]]
approve
calls useforceApprove
?transfer
calls usesafeTransfer
?transferFrom
calls usemsg.sender
as thefrom
address?call
,delegatecall
,staticcall
,transfer
,send
)success
boolean checked to handle failed calls?delegatecall
, are there strict access controls on the addresses that can be called? It shouldn't be possible todelegatecall
arbitrary addresses, so the list of possible targets should either be immutable or tightly controlled by an admin.nonReentrant
?payable
functions restricted to avoid stuck ether?Safe
functions are altered, are potential underflows and overflows caught so that a failure flag can be thrown?