Closed jalextowle closed 2 months ago
Benchmark suite | Current: 4ea19f3a6b3b6b6c183dec652769aca5ecc29b2d | Previous: e0a849d0a60c781a8fd91131d1e0dec3f4e8d498 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33937 gas |
33937 gas |
0% |
π° |
addLiquidity: avg |
197433 gas |
155512 gas |
26.9568% |
π¨ |
addLiquidity: max |
474972 gas |
429217 gas |
10.6601% |
π¨ |
checkpoint: min |
40316 gas |
40316 gas |
0% |
π° |
checkpoint: avg |
144492 gas |
144473 gas |
0.0132% |
π¨ |
checkpoint: max |
256115 gas |
255724 gas |
0.1529% |
π¨ |
closeLong: min |
31361 gas |
31361 gas |
0% |
π° |
closeLong: avg |
135909 gas |
135905 gas |
0.0029% |
π¨ |
closeLong: max |
2539376 gas |
2539376 gas |
0% |
π° |
closeShort: min |
31327 gas |
31327 gas |
0% |
π° |
closeShort: avg |
131329 gas |
131160 gas |
0.1289% |
π¨ |
closeShort: max |
271293 gas |
270901 gas |
0.1447% |
π¨ |
initialize: min |
31283 gas |
31349 gas |
-0.2105% |
β |
initialize: avg |
352639 gas |
333267 gas |
5.8128% |
π¨ |
initialize: max |
420898 gas |
399530 gas |
5.3483% |
π¨ |
openLong: min |
33437 gas |
33370 gas |
0.2008% |
π¨ |
openLong: avg |
174338 gas |
173125 gas |
0.7006% |
π¨ |
openLong: max |
333832 gas |
333607 gas |
0.0674% |
π¨ |
openShort: min |
34003 gas |
33936 gas |
0.1974% |
π¨ |
openShort: avg |
174309 gas |
168549 gas |
3.4174% |
π¨ |
openShort: max |
415072 gas |
415253 gas |
-0.0436% |
β |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
π° |
redeemWithdrawalShares: avg |
75199 gas |
75281 gas |
-0.1089% |
β |
redeemWithdrawalShares: max |
305243 gas |
305243 gas |
0% |
π° |
removeLiquidity: min |
31279 gas |
31301 gas |
-0.0703% |
β |
removeLiquidity: avg |
214954 gas |
214959 gas |
-0.0023% |
β |
removeLiquidity: max |
403632 gas |
403654 gas |
-0.0055% |
β |
This comment was automatically generated by workflow using github-action-benchmark.
Resolved Issues
Description
This PR adds a simple check to
addLiquidity()
andinitialize()
to ensure that when liquidity is added, a spot rate of 0% is still achievable.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?