Added some unchecked scopes in safe places

pooltogether · Oct 14, 2021 · 100180f · 100180f
1 parent 293e224
commit 100180f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/contracts/libraries/TwabLib.sol b/contracts/libraries/TwabLib.sol
@@ -111,7 +111,9 @@ library TwabLib {
         require(_accountDetails.balance >= _amount, _revertMessage);
 
         (accountDetails, twab, isNew) = _nextTwab(_account.twabs, _accountDetails, _currentTime);
-        accountDetails.balance = _accountDetails.balance - _amount;
+        unchecked {
+            accountDetails.balance = _accountDetails.balance - _amount;
+        }
     }
 
     /** @notice Calculates the average balance held by a user for a given time frame.

diff --git a/contracts/prize-strategy/PrizeSplit.sol b/contracts/prize-strategy/PrizeSplit.sol
@@ -72,7 +72,10 @@ abstract contract PrizeSplit is IPrizeSplit, Ownable {
 
         // Remove old prize splits configs. Match storage _prizesSplits.length with the passed newPrizeSplits.length
         while (_prizeSplits.length > newPrizeSplitsLength) {
-            uint256 _index = _prizeSplits.length - 1;
+            uint256 _index;
+            unchecked {
+                _index = _prizeSplits.length - 1;
+            }
             _prizeSplits.pop();
             emit PrizeSplitRemoved(_index);
         }