[Hacken 2022-11-25 Low #9] Unclear Use of the Virtual Specifier

[zhongfu]

There are functions in the contracts that are declared with the virtual specifier. These functions are not expected to be overridden, so the use of the virtual specifier is redundant.

Path

./src/Funnel.sol : computeDomainSeparator(), permit(), permitRenewable(), _checkOnTransferReceived(), _checkOnApprovalReceived(), supportsInterface(), _fallback()

Recommendation

Consider removing redundant code.

Status

New