The PasswordStore::s_owner
is not going to be changed by the contract. Setting it to immutable would allow to save gas.
I submit this small finding as a test as it is my first submission :)
Generally speaking, reading from storage is less efficient gas-wise than reading from bytecode. Since the variable is never going to be changed, it is a better practice to set it as immutable so that users can save gas using it.
Wastes gas.
VS Code
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.