Closed jalextowle closed 5 months ago
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/factory/HyperdriveFactory.sol | 1 | 99.16% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 8929108966: | 0.08% |
Covered Lines: | 1833 |
Relevant Lines: | 1966 |
Benchmark suite | Current: 167ead6665ae05b42918054317d8a931b2f585d5 | Previous: 1389e3b571fc11c5acfe5adb323a4e41cfedba35 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33827 gas |
33893 gas |
-0.1947% |
β |
addLiquidity: avg |
155982 gas |
148557 gas |
4.9981% |
π¨ |
addLiquidity: max |
428439 gas |
416093 gas |
2.9671% |
π¨ |
checkpoint: min |
40270 gas |
40220 gas |
0.1243% |
π¨ |
checkpoint: avg |
142260 gas |
104465 gas |
36.1796% |
π¨ |
checkpoint: max |
253402 gas |
211582 gas |
19.7654% |
π¨ |
closeLong: min |
31361 gas |
31517 gas |
-0.4950% |
β |
closeLong: avg |
135140 gas |
136932 gas |
-1.3087% |
β |
closeLong: max |
2625774 gas |
2640435 gas |
-0.5552% |
β |
closeShort: min |
31349 gas |
31394 gas |
-0.1433% |
β |
closeShort: avg |
131602 gas |
133435 gas |
-1.3737% |
β |
closeShort: max |
262932 gas |
253640 gas |
3.6635% |
π¨ |
initialize: min |
31371 gas |
31265 gas |
0.3390% |
π¨ |
initialize: avg |
330389 gas |
276137 gas |
19.6468% |
π¨ |
initialize: max |
396281 gas |
344216 gas |
15.1257% |
π¨ |
openLong: min |
33370 gas |
33348 gas |
0.0660% |
π¨ |
openLong: avg |
173394 gas |
166642 gas |
4.0518% |
π¨ |
openLong: max |
306679 gas |
253064 gas |
21.1863% |
π¨ |
openShort: min |
33936 gas |
33959 gas |
-0.0677% |
β |
openShort: avg |
167907 gas |
162950 gas |
3.0420% |
π¨ |
openShort: max |
415363 gas |
406806 gas |
2.1035% |
π¨ |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
π° |
redeemWithdrawalShares: avg |
74482 gas |
73122 gas |
1.8599% |
π¨ |
redeemWithdrawalShares: max |
209762 gas |
204973 gas |
2.3364% |
π¨ |
removeLiquidity: min |
31301 gas |
31191 gas |
0.3527% |
π¨ |
removeLiquidity: avg |
208999 gas |
198301 gas |
5.3948% |
π¨ |
removeLiquidity: max |
403574 gas |
395673 gas |
1.9969% |
π¨ |
This comment was automatically generated by workflow using github-action-benchmark.
Description
This PR removes
HyperdriveTarget4
andHyperdriveTarget5
since those targets were no longer needed after makingLPMath
an external library. In addition to removing these targets, the PR also addsgetNumberOfTargets()
to theIHyperdriveDeployerCoordinator
interface so that it is easy to figure out how many times to calldeployTarget
when deploying new instances through the factory.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 and/or Rust, the corresponding checklist can be removed.@jrhea
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?@mcclurejt
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?