fix: remove poc code

Author: ypatil12

src/contracts/core/DelegationManager.sol

@@ -529,21 +529,16 @@ contract DelegationManager is
         bool receiveAsTokens
     ) internal {
         _checkInputArrayLengths(tokens.length, withdrawal.strategies.length);
-        if (withdrawal.delegatedTo == address(this)) {
-            // If this is a redistribution withdrawal // TODO: make this cleaner
-            require(receiveAsTokens, WithdrawerNotCaller());
-        } else {
-            require(msg.sender == withdrawal.withdrawer, WithdrawerNotCaller());
-        }
+
+        require(msg.sender == withdrawal.withdrawer, WithdrawerNotCaller());
         bytes32 withdrawalRoot = calculateWithdrawalRoot(withdrawal);
         require(pendingWithdrawals[withdrawalRoot], WithdrawalNotQueued());
 
         uint256[] memory prevSlashingFactors;
         {
-            // The slashableUntil block is inclusive, so we verify that the current block number exceeds it
-            // before allowing the withdrawal to be completed.
-            uint32 delayBlocks = withdrawal.delegatedTo == address(this) ? 4 days : MIN_WITHDRAWAL_DELAY_BLOCKS;
-            uint32 slashableUntil = withdrawal.startBlock + delayBlocks;
+            // slashableUntil is block inclusive so we need to check if the current block is strictly greater than the slashableUntil block
+            // meaning the withdrawal can be completed.
+            uint32 slashableUntil = withdrawal.startBlock + MIN_WITHDRAWAL_DELAY_BLOCKS;
             require(uint32(block.number) > slashableUntil, WithdrawalDelayNotElapsed());
 
             // Given the max magnitudes of the operator the staker was originally delegated to, calculate
@@ -713,7 +708,9 @@ contract DelegationManager is
         // Emit event for operator shares being slashed
         emit OperatorSharesSlashed(operator, strategy, totalDepositSharesToBurn);
 
-        _getShareManager(strategy).increaseBurnOrRedistributableShares(operatorSet, slashId, strategy, totalDepositSharesToBurn);
+        _getShareManager(strategy).increaseBurnOrRedistributableShares(
+            operatorSet, slashId, strategy, totalDepositSharesToBurn
+        );
 
         return totalDepositSharesToBurn;
     }

src/contracts/core/StrategyManager.sol

@@ -167,7 +167,10 @@ contract StrategyManager is
     }
 
     /// @inheritdoc IStrategyManager
-    function decreaseBurnOrRedistributableShares(OperatorSet calldata operatorSet, uint256 slashId) external nonReentrant {
+    function decreaseBurnOrRedistributableShares(
+        OperatorSet calldata operatorSet,
+        uint256 slashId
+    ) external nonReentrant {
         EnumerableMap.AddressToUintMap storage burnOrRedistributableShares =
             _burnOrRedistributableShares[operatorSet.key()][slashId];
 

src/contracts/core/StrategyManagerStorage.sol

@@ -79,7 +79,8 @@ abstract contract StrategyManagerStorage is IStrategyManager {
     EnumerableMap.AddressToUintMap internal burnableShares;
 
     /// @notice Returns the amount of `shares` that have been slashed on EigenLayer but not marked for burning or redistribution yet.
-    mapping(bytes32 operatorSetKey => mapping(uint256 slashId => EnumerableMap.AddressToUintMap)) internal _burnOrRedistributableShares;
+    mapping(bytes32 operatorSetKey => mapping(uint256 slashId => EnumerableMap.AddressToUintMap)) internal
+        _burnOrRedistributableShares;
 
     // Construction
 

src/contracts/interfaces/IStrategyManager.sol

@@ -56,9 +56,7 @@ interface IStrategyManagerEvents {
 
     /// @notice Emitted when shares are burnt
     /// @dev This event is only emitted in the pre-redistribution slash path
-    event BurnableSharesDecreased(
-        IStrategy strategy, uint256 shares
-    );
+    event BurnableSharesDecreased(IStrategy strategy, uint256 shares);
 }
 
 /**