Gas Optimizations

[code423n4]

Make ProjectFactory.homeFi immutable

Code: https://github.com/code-423n4/2022-08-rigor/blob/5ab7ea84a1516cb726421ef690af5bc41029f88f/contracts/Project.sol#L52

• Project.homeFi can't be changed after init of the clone
• Since ProjectFactory.homeFi can't be changed currently, that means it's also isn't changed across clones (i.e. all clones have the same value).
• homeFi is used every tx at the Project contract, since every tx calls isTrustedForwarder()
• The cost of reading homeFi from storage is 2.1K

Therefore consider making Project.homeFi immutable. If in the future you decide to make it modifiable, just change ProjectFactory.underlying contract and make it a storage var (or create the contract with a different value and keep it immutable).

Cache storage variable at ProjectFactory.createProject()

Code: https://github.com/code-423n4/2022-08-rigor/blob/5ab7ea84a1516cb726421ef690af5bc41029f88f/contracts/ProjectFactory.sol#L78-L93

Instead of reading homeFi from storage twice, cache it to memory. This will save ~100 gas units.

         returns (address _clone)
     {
+        address _homeFi = homeFi;
         // Revert if sender is not HomeFi
-        require(_msgSender() == homeFi, "PF::!HomeFiContract");
+        require(_msgSender() == _homeFi, "PF::!HomeFiContract");
 
         // Create clone of Project implementation
         _clone = ClonesUpgradeable.clone(underlying);
 
         // Initialize project
-        Project(_clone).initialize(_currency, _sender, homeFi);
+        Project(_clone).initialize(_currency, _sender, _homeFi);
     }