use values() method of EnumerableSet when possible, to make the code simpler

hoytech · hoytech · commit c751556a304f · 2025-04-29T14:36:42.000-04:00
diff --git a/src/EulerSwapFactory.sol b/src/EulerSwapFactory.sol
@@ -119,7 +119,7 @@ contract EulerSwapFactory is IEulerSwapFactory, EVCUtil, ProtocolFee {
 
     /// @inheritdoc IEulerSwapFactory
     function pools() external view returns (address[] memory) {
-        return _getSlice(allPools, 0, type(uint256).max);
+        return allPools.values();
     }
 
     /// @inheritdoc IEulerSwapFactory
@@ -138,7 +138,7 @@ contract EulerSwapFactory is IEulerSwapFactory, EVCUtil, ProtocolFee {
 
     /// @inheritdoc IEulerSwapFactory
     function poolsByPair(address asset0, address asset1) external view returns (address[] memory) {
-        return _getSlice(poolMap[asset0][asset1], 0, type(uint256).max);
+        return poolMap[asset0][asset1].values();
     }
 
     /// @notice Validates operator authorization for euler account and update the relevant EulerAccountState.

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ contract EulerSwapFactory is IEulerSwapFactory, EVCUtil, ProtocolFee {`
`119`	`119`
`120`	`120`	`/// @inheritdoc IEulerSwapFactory`
`121`	`121`	`function pools() external view returns (address[] memory) {`
`122`		`- return _getSlice(allPools, 0, type(uint256).max);`
	`122`	`+ return allPools.values();`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`/// @inheritdoc IEulerSwapFactory`
`@@ -138,7 +138,7 @@ contract EulerSwapFactory is IEulerSwapFactory, EVCUtil, ProtocolFee {`
`138`	`138`
`139`	`139`	`/// @inheritdoc IEulerSwapFactory`
`140`	`140`	`function poolsByPair(address asset0, address asset1) external view returns (address[] memory) {`
`141`		`- return _getSlice(poolMap[asset0][asset1], 0, type(uint256).max);`
	`141`	`+ return poolMap[asset0][asset1].values();`
`142`	`142`	`}`
`143`	`143`
`144`	`144`	`/// @notice Validates operator authorization for euler account and update the relevant EulerAccountState.`