chore: clean up

Author: jaybuidl

contracts/src/arbitration/KlerosCoreBase.sol

@@ -1160,8 +1160,6 @@ abstract contract KlerosCoreBase is IArbitratorV2, Initializable, UUPSProxiable
     error CannotDisableClassicDK();
     error StakingNotPossibleInThisCourt();
     error StakingLessThanCourtMinStake();
-    error StakingTransferFailed();
-    error UnstakingTransferFailed();
     error ArbitrationFeesNotEnough();
     error DisputeKitNotSupportedByCourt();
     error MustSupportDisputeKitClassic();

contracts/src/arbitration/StakeControllerBase.sol

@@ -413,10 +413,11 @@ abstract contract StakeControllerBase is IStakeController, Initializable, UUPSPr
             }
         }
 
+        // If the courtID was found in the array
         if (courtIndexToRemove != type(uint256).max) {
-            // If the courtID was found in the array
-            // If it's not the last element, swap the last element into its place
+            // If it's not the last element
             if (courtIndexToRemove != length - 1) {
+                // Swap the last element into its place
                 _stakedCourts[courtIndexToRemove] = _stakedCourts[length - 1];
             }
             // Remove the last element (either the original last, or the one that was swapped)
@@ -440,5 +441,4 @@ abstract contract StakeControllerBase is IStakeController, Initializable, UUPSPr
     error NotInStakingPhase();
     error NotDrawingPhase();
     error NoDelayedStakes();
-    error InvalidMigrationData();
 }

contracts/src/libraries/Constants.sol

@@ -20,7 +20,7 @@ uint256 constant DEFAULT_K = 6; // Default number of children per node.
 uint256 constant DEFAULT_NB_OF_JURORS = 3; // The default number of jurors in a dispute.
 IERC20 constant NATIVE_CURRENCY = IERC20(address(0)); // The native currency, such as ETH on Arbitrum, Optimism and Ethereum L1.
 
-// DEPRECATED
+// DEPRECATED: still used by University contracts for now
 enum OnError {
     Revert,
     Return
@@ -29,7 +29,7 @@ enum OnError {
 enum StakingResult {
     Successful,
     Delayed,
-    StakingTransferFailed, // DEPRECATED
+    StakingTransferFailed, // DEPRECATED: still used by University contracts for now
     UnstakingTransferFailed, // DEPRECATED
     CannotStakeInMoreCourts, // DEPRECATED
     CannotStakeInThisCourt, // DEPRECATED