Remove unused return names

Uniswap · Aug 24, 2023 · 5e5ef41 · 5e5ef41
1 parent 6eae896
commit 5e5ef41
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/.forge-snapshots/Base-DutchOrder-BaseExecuteSingleWithFee.snap b/.forge-snapshots/Base-DutchOrder-BaseExecuteSingleWithFee.snap
@@ -1 +1 @@
-181972
+150411
diff --git a/.forge-snapshots/Base-ExclusiveDutchOrder-BaseExecuteSingleWithFee.snap b/.forge-snapshots/Base-ExclusiveDutchOrder-BaseExecuteSingleWithFee.snap
@@ -1 +1 @@
-150955
+182180
diff --git a/.forge-snapshots/Base-LimitOrderReactor-BaseExecuteSingleWithFee.snap b/.forge-snapshots/Base-LimitOrderReactor-BaseExecuteSingleWithFee.snap
@@ -1 +1 @@
-146882
+178116
diff --git a/src/lens/OrderQuoter.sol b/src/lens/OrderQuoter.sol
@@ -43,7 +43,7 @@ contract OrderQuoter is IReactorCallback {
     /// @notice Return the order info of a given order (abi-encoded bytes).
     /// @param reason The revert reason
     /// @return order abi-encoded order, including `reactor` as the first encoded struct member
-    function parseRevertReason(bytes memory reason) private pure returns (ResolvedOrder memory order) {
+    function parseRevertReason(bytes memory reason) private pure returns (ResolvedOrder memory) {
         if (reason.length < RESOLVED_ORDER_MIN_LENGTH) {
             assembly {
                 revert(add(32, reason), mload(reason))

diff --git a/src/lib/ExclusivityOverrideLib.sol b/src/lib/ExclusivityOverrideLib.sol
@@ -54,7 +54,7 @@ library ExclusivityOverrideLib {
     /// @dev if the order has no exclusivity, always returns true
     /// @dev if the order has active exclusivity and the current filler is the exclusive address, returns true
     /// @dev if the order has active exclusivity and the current filler is not the exclusive address, returns false
-    function hasFillingRights(address exclusive, uint256 exclusivityEndTime) internal view returns (bool pass) {
+    function hasFillingRights(address exclusive, uint256 exclusivityEndTime) internal view returns (bool) {
         return exclusive == address(0) || block.timestamp > exclusivityEndTime || exclusive == msg.sender;
     }
 }