Skip to content

Commit 56472a7

Browse files
committed
Use non-named function argument if not used
1 parent 5dbf505 commit 56472a7

File tree

2 files changed

+3
-12
lines changed

2 files changed

+3
-12
lines changed

src/Aggor.sol

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -423,19 +423,13 @@ contract Aggor is IAggor, IToll, Auth {
423423

424424
/// @inheritdoc IToll
425425
/// @dev Function is disabled!
426-
function kiss(address who) external view auth {
427-
// No-op to silence compiler warnings.
428-
require(who == address(0) || who != address(0));
429-
426+
function kiss(address /*who*/ ) external view auth {
430427
revert();
431428
}
432429

433430
/// @inheritdoc IToll
434431
/// @dev Function is disabled!
435-
function diss(address who) external view auth {
436-
// No-op to silence compiler warnings.
437-
require(who == address(0) || who != address(0));
438-
432+
function diss(address /*who*/ ) external view auth {
439433
revert();
440434
}
441435

test/mocks/UniswapPoolMock.sol

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,14 +50,11 @@ contract UniswapPoolMock {
5050
shouldOverflowUint128 = shouldOverflowUint128_;
5151
}
5252

53-
function observe(uint32[] memory secondsAgo)
53+
function observe(uint32[] memory /*secondsAgo*/ )
5454
public
5555
view
5656
returns (int56[] memory, uint160[] memory)
5757
{
58-
// No-op to silence compiler warnings.
59-
require(secondsAgo.length != 0 || secondsAgo.length == 0);
60-
6158
int56[] memory tickCumulatives = new int56[](2);
6259
uint160[] memory secondsPerLiquidityCumulativeX128s = new uint160[](2);
6360

0 commit comments

Comments
 (0)