chore: update certain parts

Author: ypatil12

src/contracts/core/DelegationManager.sol

@@ -96,7 +96,7 @@ contract DelegationManager is
     function registerAsOperator(
         address initDelegationApprover,
         uint32 allocationDelay,
-        string memory metadataURI
+        string calldata metadataURI
     ) external nonReentrant {
         require(!isDelegated(msg.sender), ActivelyDelegated());
 
@@ -120,7 +120,7 @@ contract DelegationManager is
     }
 
     /// @inheritdoc IDelegationManager
-    function updateOperatorMetadataURI(address operator, string memory metadataURI) external checkCanCall(operator) {
+    function updateOperatorMetadataURI(address operator, string calldata metadataURI) external checkCanCall(operator) {
         require(isOperator(operator), OperatorNotRegistered());
         emit OperatorMetadataURIUpdated(operator, metadataURI);
     }
@@ -179,7 +179,7 @@ contract DelegationManager is
 
     /// @inheritdoc IDelegationManager
     function queueWithdrawals(
-        QueuedWithdrawalParams[] memory params
+        QueuedWithdrawalParams[] calldata params
     ) external onlyWhenNotPaused(PAUSED_ENTER_WITHDRAWAL_QUEUE) nonReentrant returns (bytes32[] memory) {
         bytes32[] memory withdrawalRoots = new bytes32[](params.length);
         address operator = delegatedTo[msg.sender];
@@ -207,18 +207,18 @@ contract DelegationManager is
 
     /// @inheritdoc IDelegationManager
     function completeQueuedWithdrawal(
-        Withdrawal memory withdrawal,
-        IERC20[] memory tokens,
+        Withdrawal calldata withdrawal,
+        IERC20[] calldata tokens,
         bool receiveAsTokens
     ) external onlyWhenNotPaused(PAUSED_EXIT_WITHDRAWAL_QUEUE) nonReentrant {
         _completeQueuedWithdrawal(withdrawal, tokens, receiveAsTokens);
     }
 
     /// @inheritdoc IDelegationManager
     function completeQueuedWithdrawals(
-        Withdrawal[] memory withdrawals,
-        IERC20[][] memory tokens,
-        bool[] memory receiveAsTokens
+        Withdrawal[] calldata withdrawals,
+        IERC20[][] calldata tokens,
+        bool[] calldata receiveAsTokens
     ) external onlyWhenNotPaused(PAUSED_EXIT_WITHDRAWAL_QUEUE) nonReentrant {
         uint256 n = withdrawals.length;
         for (uint256 i; i < n; ++i) {
@@ -529,7 +529,7 @@ contract DelegationManager is
 
         pendingWithdrawals[withdrawalRoot] = true;
         _queuedWithdrawals[withdrawalRoot] = withdrawal;
-        _stakerQueuedWithdrawalRoots[withdrawal.staker].add(withdrawalRoot);
+        _stakerQueuedWithdrawalRoots[staker].add(withdrawalRoot);
 
         emit SlashingWithdrawalQueued(withdrawalRoot, withdrawal, withdrawableShares);
         return withdrawalRoot;
@@ -546,7 +546,7 @@ contract DelegationManager is
      */
     function _completeQueuedWithdrawal(
         Withdrawal memory withdrawal,
-        IERC20[] memory tokens,
+        IERC20[] calldata tokens,
         bool receiveAsTokens
     ) internal {
         require(tokens.length == withdrawal.strategies.length, InputArrayLengthMismatch());
@@ -688,7 +688,7 @@ contract DelegationManager is
     }
 
     /// @dev If `operator` has configured a `delegationApprover`, check that `signature` and `salt`
-    /// are a valid appfroval for `staker` delegating to `operator`.
+    /// are a valid approval for `staker` delegating to `operator`.
     function _checkApproverSignature(
         address staker,
         address operator,
@@ -1063,4 +1063,4 @@ contract DelegationManager is
             )
         );
     }
-}
+}

src/contracts/interfaces/IDelegationManager.sol

@@ -174,9 +174,6 @@ interface IDelegationManagerEvents is IDelegationManagerTypes {
 
     /// @notice Emitted whenever an operator's shares are slashed for a given strategy
     event OperatorSharesSlashed(address indexed operator, IStrategy strategy, uint256 totalSlashedShares);
-
-    /// @notice Emitted when a redistribution is queued
-    event RedistributionQueued(bytes32 withdrawalRoot, Withdrawal withdrawal);
 }
 
 /**
@@ -570,4 +567,4 @@ interface IDelegationManager is ISignatureUtilsMixin, IDelegationManagerErrors,
 
     /// @notice The EIP-712 typehash for the DelegationApproval struct used by the contract
     function DELEGATION_APPROVAL_TYPEHASH() external view returns (bytes32);
-}
+}

src/test/unit/StrategyManagerUnit.t.sol

@@ -1336,9 +1336,9 @@ contract StrategyManagerUnitTests_clearBurnOrRedistributableShares is StrategyMa
         emit BurnOrRedistributableSharesIncreased(defaultOperatorSet, defaultSlashId, strategy, sharesToRemove);
         strategyManager.increaseBurnOrRedistributableShares(defaultOperatorSet, defaultSlashId, strategy, sharesToRemove);
 
-//     // Now set token to be contract that reverts simulating an upgrade
-//     cheats.etch(address(token), address(revertToken).code);
-//     ERC20_SetTransferReverting_Mock(address(token)).setTransfersRevert(true);
+        // Now set token to be contract that reverts simulating an upgrade
+        cheats.etch(address(token), address(revertToken).code);
+        ERC20_SetTransferReverting_Mock(address(token)).setTransfersRevert(true);
 
         cheats.expectRevert("SafeERC20: low-level call failed");
         cheats.prank(address(delegationManagerMock));
@@ -1509,4 +1509,4 @@ contract StrategyManagerUnitTests_removeStrategiesFromDepositWhitelist is Strate
             }
         }
     }
-}
+}