chore: minor comments tweaks

Author: jaybuidl

contracts/src/arbitration/KlerosCoreBase.sol

@@ -1079,9 +1079,10 @@ abstract contract KlerosCoreBase is IArbitratorV2, Initializable, UUPSProxiable
  courtJump = true;
 
  if (!courts[newCourtID].supportedDisputeKits[newDisputeKitID]) {
+ // The current Dispute Kit is not compatible with the new court, jump to another Dispute Kit.
  newDisputeKitID = disputeKits[_round.disputeKitID].getJumpDisputeKitID();
  if (!courts[newCourtID].supportedDisputeKits[newDisputeKitID]) {
- // Switch to classic dispute kit if parent court doesn't support the jump DK.
+ // The new Dispute Kit is still not compatible, fall back to `DisputeKitClassic` which is always supported.
  newDisputeKitID = DISPUTE_KIT_CLASSIC;
  }
  disputeKitJump = true;

contracts/src/arbitration/dispute-kits/DisputeKitClassicBase.sol

@@ -180,18 +180,18 @@ abstract contract DisputeKitClassicBase is IDisputeKit, Initializable, UUPSProxi
  owner = _owner;
  }
 
- /// @dev Changes the dispute kit ID used for the jump.
- /// @param _jumpDisputeKitID The new value for the `jumpDisputeKitID` storage variable.
- function changeJumpDisputeKitID(uint256 _jumpDisputeKitID) external onlyByOwner {
- jumpDisputeKitID = _jumpDisputeKitID;
- }
-
  /// @dev Changes the `core` storage variable.
  /// @param _core The new value for the `core` storage variable.
  function changeCore(address _core) external onlyByOwner {
  core = KlerosCore(_core);
  }
 
+ /// @dev Changes the dispute kit ID used for the jump.
+ /// @param _jumpDisputeKitID The new value for the `jumpDisputeKitID` storage variable.
+ function changeJumpDisputeKitID(uint256 _jumpDisputeKitID) external onlyByOwner {
+ jumpDisputeKitID = _jumpDisputeKitID;
+ }
+
  // ************************************* //
  // * State Modifiers * //
  // ************************************* //
@@ -649,7 +649,9 @@ abstract contract DisputeKitClassicBase is IDisputeKit, Initializable, UUPSProxi
  return (_currentNbVotes * 2) + 1;
  }
 
- function getJumpDisputeKitID() external view returns (uint256) {
+ /// @dev Returns the dispute kid ID be used after court jump by Kleros Core.
+ /// @return The ID of the dispute kit in Kleros Core disputeKits array.
+ function getJumpDisputeKitID() external view override returns (uint256) {
  // Fall back to classic DK in case the jump ID is not defined.
  return jumpDisputeKitID == 0 ? DISPUTE_KIT_CLASSIC : jumpDisputeKitID;
  }

contracts/src/arbitration/interfaces/IDisputeKit.sol

@@ -127,7 +127,7 @@ interface IDisputeKit {
  uint256 _currentNbVotes
  ) external view returns (uint256); // TODO: remove previousDisputeKit
 
- /// @dev Returns the dispute kid ID for Kleros Core, that will be used after court jump.
+ /// @dev Returns the dispute kid ID be used after court jump by Kleros Core.
  /// @return The ID of the dispute kit in Kleros Core disputeKits array.
  function getJumpDisputeKitID() external view returns (uint256);