Skip to content

Commit 16ed9ef

Browse files
committed
remove unused returndata vars
1 parent df4ea03 commit 16ed9ef

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

packages/contracts-bedrock/test/L1/OPContractsManager.t.sol

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ contract OPContractsManager_Upgrade_Harness is CommonTest, DisputeGames {
232232
}
233233

234234
vm.prank(_delegateCaller, true);
235-
(bool upgradeSuccess, bytes memory upgradeReturndata) =
235+
(bool upgradeSuccess,) =
236236
address(_opcm).delegatecall(abi.encodeCall(IOPContractsManager.upgrade, (opChainConfigs)));
237237
assertTrue(upgradeSuccess, "upgrade failed");
238238

@@ -1189,7 +1189,7 @@ contract OPContractsManager_UpdatePrestate_Test is OPContractsManager_TestInit {
11891189
// Trigger the updatePrestate function.
11901190
address proxyAdminOwner = chainDeployOutput1.opChainProxyAdmin.owner();
11911191
vm.prank(proxyAdminOwner, true);
1192-
(bool success, bytes memory returndata) =
1192+
(bool success,) =
11931193
address(prestateUpdater).delegatecall(abi.encodeCall(IOPContractsManager.updatePrestate, (inputs)));
11941194
assertTrue(success, "updatePrestate failed");
11951195

@@ -1317,7 +1317,7 @@ contract OPContractsManager_UpdatePrestate_Test is OPContractsManager_TestInit {
13171317
// Trigger the updatePrestate function.
13181318
address proxyAdminOwner = chainDeployOutput1.opChainProxyAdmin.owner();
13191319
vm.prank(proxyAdminOwner, true);
1320-
(bool success, bytes memory returndata) =
1320+
(bool success,) =
13211321
address(prestateUpdater).delegatecall(abi.encodeCall(IOPContractsManager.updatePrestate, (inputs)));
13221322
assertTrue(success, "updatePrestate failed");
13231323

@@ -1726,7 +1726,7 @@ contract OPContractsManager_UpgradeSuperchainConfig_Test is OPContractsManager_U
17261726
vm.expectEmit(address(superchainConfig));
17271727
emit Upgraded(impls.superchainConfigImpl);
17281728
vm.prank(superchainPAO, true);
1729-
(bool success, bytes memory returndata) =
1729+
(bool success,) =
17301730
address(opcm).delegatecall(abi.encodeCall(IOPContractsManager.upgradeSuperchainConfig, (superchainConfig)));
17311731
assertTrue(success, "upgradeSuperchainConfig failed");
17321732
}
@@ -1752,7 +1752,7 @@ contract OPContractsManager_UpgradeSuperchainConfig_Test is OPContractsManager_U
17521752

17531753
vm.expectRevert("Ownable: caller is not the owner");
17541754
vm.prank(delegateCaller, true);
1755-
(bool success, bytes memory returndata) =
1755+
(bool success,) =
17561756
address(opcm).delegatecall(abi.encodeCall(IOPContractsManager.upgradeSuperchainConfig, (superchainConfig)));
17571757
assertTrue(success, "upgradeSuperchainConfig failed");
17581758
}
@@ -1772,7 +1772,7 @@ contract OPContractsManager_UpgradeSuperchainConfig_Test is OPContractsManager_U
17721772
vm.etch(upgrader, hex"00"); // Foundry fails with "cannot `prank` delegate call from an EOA" if empty
17731773
vm.expectRevert(IOPContractsManagerUpgrader.OPContractsManagerUpgrader_SuperchainConfigAlreadyUpToDate.selector);
17741774
vm.prank(upgrader, true);
1775-
(bool success, bytes memory returndata) =
1775+
(bool success,) =
17761776
address(opcm).delegatecall(abi.encodeCall(IOPContractsManager.upgradeSuperchainConfig, (superchainConfig)));
17771777
assertTrue(success, "upgradeSuperchainConfig failed");
17781778
}
@@ -1847,8 +1847,7 @@ contract OPContractsManager_Migrate_Test is OPContractsManager_TestInit {
18471847
vm.expectRevert(_revertSelector);
18481848
}
18491849
vm.prank(proxyAdminOwner, true);
1850-
(bool success, bytes memory returndata) =
1851-
address(opcm).delegatecall(abi.encodeCall(IOPContractsManager.migrate, (_input)));
1850+
(bool success,) = address(opcm).delegatecall(abi.encodeCall(IOPContractsManager.migrate, (_input)));
18521851
assertTrue(success, "migrate failed");
18531852
uint256 gasAfter = gasleft();
18541853

0 commit comments

Comments
 (0)