refactor: review changes (#1411)

**Motivation:**

Minor nits.

**Modifications:**

- Add `slashOperator` return value natspec.
- Remove `pause/unpauseRedistribution` codesize optimization.

**Result:**

Cleaner and shorter diff for audit.

Author: 0xClandestine

src/contracts/core/SlashEscrowFactory.sol

@@ -117,15 +117,15 @@ contract SlashEscrowFactory is Initializable, SlashEscrowFactoryStorage, Ownable
      */
 
     /// @inheritdoc ISlashEscrowFactory
-    function pauseEscrow(OperatorSet calldata operatorSet, uint256 slashId) external onlyPauser {
-        _checkNewPausedStatus(operatorSet, slashId, true);
+    function pauseEscrow(OperatorSet calldata operatorSet, uint256 slashId) external virtual onlyPauser {
+        require(!_paused[operatorSet.key()][slashId], IPausable.InvalidNewPausedStatus());
         _paused[operatorSet.key()][slashId] = true;
         emit EscrowPaused(operatorSet, slashId);
     }
 
     /// @inheritdoc ISlashEscrowFactory
-    function unpauseEscrow(OperatorSet calldata operatorSet, uint256 slashId) external onlyUnpauser {
-        _checkNewPausedStatus(operatorSet, slashId, false);
+    function unpauseEscrow(OperatorSet calldata operatorSet, uint256 slashId) external virtual onlyUnpauser {
+        require(_paused[operatorSet.key()][slashId], IPausable.InvalidNewPausedStatus());
         _paused[operatorSet.key()][slashId] = false;
         emit EscrowUnpaused(operatorSet, slashId);
     }
@@ -208,16 +208,6 @@ contract SlashEscrowFactory is Initializable, SlashEscrowFactoryStorage, Ownable
         emit GlobalEscrowDelaySet(delay);
     }
 
-    /// @notice Checks that the new paused status is not the same as the current paused status.
-    /// @dev This is needed for event sanitization.
-    function _checkNewPausedStatus(
-        OperatorSet calldata operatorSet,
-        uint256 slashId,
-        bool newPauseStatus
-    ) internal view {
-        require(_paused[operatorSet.key()][slashId] != newPauseStatus, IPausable.InvalidNewPausedStatus());
-    }
-
     /**
      * @notice Deploys a `SlashEscrow`
      * @param operatorSet The operator set whose slash escrow is being deployed.
@@ -363,7 +353,7 @@ contract SlashEscrowFactory is Initializable, SlashEscrowFactoryStorage, Ownable
 
     /// @inheritdoc ISlashEscrowFactory
     function isEscrowPaused(OperatorSet calldata operatorSet, uint256 slashId) public view returns (bool) {
-        return _paused[operatorSet.key()][slashId];
+        return _paused[operatorSet.key()][slashId] || paused(PAUSED_RELEASE_ESCROW);
     }
 
     /// @inheritdoc ISlashEscrowFactory

src/contracts/interfaces/IAllocationManager.sol

@@ -250,6 +250,8 @@ interface IAllocationManager is IAllocationManagerErrors, IAllocationManagerEven
      * @dev Small slashing amounts may not result in actual token burns due to
      *      rounding, which will result in small amounts of tokens locked in the contract
      *      rather than fully burning through the burn mechanism.
+     * @return slashId The operator set's unique identifier for the slash.
+     * @return shares The number of shares to be burned or redistributed for each strategy that was slashed.
      */
     function slashOperator(
         address avs,