From bfc46daaddf16a1e72e86bc82f8a1cdccf7eec13 Mon Sep 17 00:00:00 2001 From: David Racero Date: Tue, 15 Sep 2020 14:25:00 +0200 Subject: [PATCH] Change modifier to function to save code size --- contracts/lendingpool/LendingPool.sol | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/contracts/lendingpool/LendingPool.sol b/contracts/lendingpool/LendingPool.sol index ba5e7894..2c9ad98a 100644 --- a/contracts/lendingpool/LendingPool.sol +++ b/contracts/lendingpool/LendingPool.sol @@ -59,12 +59,11 @@ contract LendingPool is VersionedInitializable, ILendingPool { /** * @dev only lending pools configurator can use functions affected by this modifier **/ - modifier onlyLendingPoolConfigurator { + function onlyLendingPoolConfigurator() internal view { require( _addressesProvider.getLendingPoolConfigurator() == msg.sender, Errors.CALLER_NOT_LENDING_POOL_CONFIGURATOR ); - _; } uint256 public constant UINT_MAX_VALUE = uint256(-1); @@ -431,7 +430,6 @@ contract LendingPool is VersionedInitializable, ILendingPool { uint256 purchaseAmount, bool receiveAToken ) external override { - address liquidationManager = _addressesProvider.getLendingPoolLiquidationManager(); //solium-disable-next-line @@ -791,7 +789,8 @@ contract LendingPool is VersionedInitializable, ILendingPool { address stableDebtAddress, address variableDebtAddress, address interestRateStrategyAddress - ) external override onlyLendingPoolConfigurator { + ) external override { + onlyLendingPoolConfigurator(); _reserves[asset].init( aTokenAddress, stableDebtAddress, @@ -810,16 +809,13 @@ contract LendingPool is VersionedInitializable, ILendingPool { function setReserveInterestRateStrategyAddress(address asset, address rateStrategyAddress) external override - onlyLendingPoolConfigurator { + onlyLendingPoolConfigurator(); _reserves[asset].interestRateStrategyAddress = rateStrategyAddress; } - function setConfiguration(address asset, uint256 configuration) - external - override - onlyLendingPoolConfigurator - { + function setConfiguration(address asset, uint256 configuration) external override { + onlyLendingPoolConfigurator(); _reserves[asset].configuration.data = configuration; }