From 875e4d53ecaa06872efa4e88a58b2d42c89ff7f8 Mon Sep 17 00:00:00 2001
From: Mark Spanbroek <mark@spanbroek.net>
Date: Thu, 13 Feb 2025 09:34:17 +0100
Subject: [PATCH] change constructor param to config

Changes the Marketplace constructor parameter `configuration` to `config` to prevent overshadowing the `configuration()` method.
---
 contracts/Marketplace.sol | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/contracts/Marketplace.sol b/contracts/Marketplace.sol
index 8a0c540..1a799fe 100644
--- a/contracts/Marketplace.sol
+++ b/contracts/Marketplace.sol
@@ -95,28 +95,24 @@ contract Marketplace is SlotReservations, Proofs, StateRetrieval, Endian {
   }
 
   constructor(
-    MarketplaceConfig memory configuration,
+    MarketplaceConfig memory config,
     IERC20 token_,
     IGroth16Verifier verifier
-  )
-    SlotReservations(configuration.reservations)
-    Proofs(configuration.proofs, verifier)
-  {
+  ) SlotReservations(config.reservations) Proofs(config.proofs, verifier) {
     _token = token_;
 
-    if (configuration.collateral.repairRewardPercentage > 100)
+    if (config.collateral.repairRewardPercentage > 100)
       revert Marketplace_RepairRewardPercentageTooHigh();
-    if (configuration.collateral.slashPercentage > 100)
+    if (config.collateral.slashPercentage > 100)
       revert Marketplace_SlashPercentageTooHigh();
 
     if (
-      configuration.collateral.maxNumberOfSlashes *
-        configuration.collateral.slashPercentage >
+      config.collateral.maxNumberOfSlashes * config.collateral.slashPercentage >
       100
     ) {
       revert Marketplace_MaximumSlashingTooHigh();
     }
-    _config = configuration;
+    _config = config;
   }
 
   function configuration() public view returns (MarketplaceConfig memory) {