Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deth_ctf : The variable of Vault:owner should be declared as immutable #43

Open
hrmneffdii opened this issue Aug 13, 2024 · 1 comment

Comments

@hrmneffdii
Copy link

Severity : Informational

Description

State variables that is Vault:owner, it is not updated following functionality or after deployment. It should be declared as immutable variable to save more gas.

Recommendation

Add the immutable syntax to the variable Vault:owner

-      address public owner;
+     address public immutable i_owner;
       . . . 

      constructor(address _owner){
-     owner     = _owner;
+    i_owner  = _owner;
     }
@0xdeth
Copy link
Contributor

0xdeth commented Aug 17, 2024

Informational.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants