You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ERC20Permit and Votes both implement separate DOMAIN_SEPARATOR functions that are not virtual, so they can't be combined.
In the 4.x releases this issue shouldn't show up because users would combine ERC20Permit with ERC20Votes (and not Votes), which do not present an incompatibility.
However, in the current 5.0 code, following #3816, ERC20Votes is built on Votes and this will be an issue.
#3872 proposed making DOMAIN_SEPARATOR virtual, but this is not a good use of virtual functions because they would not be chained using super. I believe we should look for another solution.
The text was updated successfully, but these errors were encountered:
Reported initially in #3872.
ERC20Permit
andVotes
both implement separateDOMAIN_SEPARATOR
functions that are not virtual, so they can't be combined.In the 4.x releases this issue shouldn't show up because users would combine
ERC20Permit
withERC20Votes
(and notVotes
), which do not present an incompatibility.However, in the current 5.0 code, following #3816,
ERC20Votes
is built onVotes
and this will be an issue.#3872 proposed making
DOMAIN_SEPARATOR
virtual, but this is not a good use of virtual functions because they would not be chained usingsuper
. I believe we should look for another solution.The text was updated successfully, but these errors were encountered: