NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use immutable variables where possible (#2528)

Browse Source 
Co-authored-by: rotciv <victorfage@gmail.com>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
This commit is contained in:
Francisco Giordano
2021-02-19 11:11:54 -03:00
committed by GitHub
parent 0059b17dfc
commit 7d20d0e2d2
4 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
contracts/payment/PullPayment.sol
View File

@ -23,7 +23,7 @@ import "./escrow/Escrow.sol";
* payments with {payments}, and retrieve them with {withdrawPayments}.
*/
abstract contract PullPayment {
Escrow private _escrow;
Escrow immutable private _escrow;
constructor () {
_escrow = new Escrow();

2
contracts/payment/escrow/RefundEscrow.sol
View File

@ -23,7 +23,7 @@ contract RefundEscrow is ConditionalEscrow {
event RefundsEnabled();
State private _state;
address payable private _beneficiary;
address payable immutable private _beneficiary;
/**
* @dev Constructor.

2
contracts/token/ERC20/ERC20Capped.sol
View File

@ -8,7 +8,7 @@ import "./ERC20.sol";
* @dev Extension of {ERC20} that adds a cap to the supply of tokens.
*/
abstract contract ERC20Capped is ERC20 {
uint256 private _cap;
uint256 immutable private _cap;
/**
* @dev Sets the value of the `cap`. This value is immutable, it can only be

6
contracts/token/ERC20/TokenTimelock.sol
View File

@ -15,13 +15,13 @@ contract TokenTimelock {
using SafeERC20 for IERC20;
// ERC20 basic token contract being held
IERC20 private _token;
IERC20 immutable private _token;
// beneficiary of tokens after they are released
address private _beneficiary;
address immutable private _beneficiary;
// timestamp when token release is enabled
uint256 private _releaseTime;
uint256 immutable private _releaseTime;
constructor (IERC20 token_, address beneficiary_, uint256 releaseTime_) {
// solhint-disable-next-line not-rely-on-time