Redundant modifiers in ERC721 safeTransferFrom

[abandeali1]

🎉 Description

• 🐛 This is a bug report.
• 📈 This is a feature request.

Both versions of safeTransferFrom have the canTransfer modifier, even though they call transferFrom under the hood (which also includes the canTransfer modifier).

📝 Details

While these extra modifiers may help with readability, they are redundant and highly inefficient. Initial tests show that removing these modifiers saves ~1800 gas for a call of safeTransferFrom with data (likely higher gas savings when calling the version without data, since that adds a third canTransfer check).