Removes annoying warnings about unused variables.

rmrk-team · Jan 4, 2024 · 4d6e16a · 4d6e16a
1 parent e249890
commit 4d6e16a
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 16 deletions.
diff --git a/contracts/RMRK/extension/RMRKRoyalties.sol b/contracts/RMRK/extension/RMRKRoyalties.sol
@@ -84,7 +84,7 @@ abstract contract RMRKRoyalties {
         uint256 tokenId,
         uint256 salePrice
     ) external view virtual returns (address receiver, uint256 royaltyAmount) {
-        // tokenId is ignored as the royalty is the same for all tokens
+        uint256(tokenId); // Silence the warning about unused variable, needed for docs generation
         receiver = _royaltyRecipient;
         royaltyAmount = (salePrice * _royaltyPercentageBps) / 10000;
     }

diff --git a/contracts/RMRK/extension/soulbound/RMRKSoulboundAfterTransactions.sol b/contracts/RMRK/extension/soulbound/RMRKSoulboundAfterTransactions.sol
@@ -47,7 +47,7 @@ abstract contract RMRKSoulboundAfterTransactions is RMRKSoulbound {
         address to,
         uint256 tokenId
     ) internal virtual {
-        // to
+        address(to); // Silence the warning about unused variable, needed for docs generation
         // We won't count minting:
         if (from != address(0)) {
             _transfersPerToken[tokenId]++;

diff --git a/contracts/mocks/OwnableMintableERC721Mock.sol b/contracts/mocks/OwnableMintableERC721Mock.sol
@@ -5,11 +5,11 @@ pragma solidity ^0.8.21;
 /// @dev This mock smart contract is intended to be used with `@defi-wonderland/smock` and doesn't need any business
 ///  logic.
 contract OwnableMintableERC721Mock {
-    function owner() public returns (address) {
+    function owner() public pure returns (address) {
         return address(0);
     }
 
-    function ownerOf(uint256 tokenId) public returns (address) {
+    function ownerOf(uint256) public pure returns (address) {
         return address(0);
     }
 }
diff --git a/docs/console.md b/docs/console.md