Skip to content

Commit 71ff0c4

Browse files
committed
revert virtual changes
1 parent 4b38ece commit 71ff0c4

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

GUIDELINES.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,5 +115,3 @@ In addition to the official Solidity Style Guide we have a number of other conve
115115
```
116116

117117
* Unchecked arithmetic blocks should contain comments explaining why overflow is guaranteed not to happen. If the reason is immediately apparent from the line above the unchecked block, the comment may be omitted.
118-
119-
* Functions should be by default `virtual`, because users frequently want to customize some behaviors. Functions that should not be virtual are those that are aliases of another function, such that any overrides to it should instead be done in the function that is aliased, usually a more general function.

contracts/token/ERC721/ERC721.sol

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,8 @@ contract ERC721 is Context, ERC165, IERC721, IERC721Metadata {
6464

6565
/**
6666
* @dev See {IERC721-ownerOf}.
67-
*
68-
* This function wraps {_ownerOf}. Any desired overrides must go there.
6967
*/
70-
function ownerOf(uint256 tokenId) public view override returns (address) {
68+
function ownerOf(uint256 tokenId) public view virtual override returns (address) {
7169
address owner = _ownerOf(tokenId);
7270
require(owner != address(0), "ERC721: invalid token ID");
7371
return owner;

0 commit comments

Comments
 (0)