Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overloaded function names not displayed as expected #138

Closed
dmuhs opened this issue Aug 9, 2023 · 1 comment · Fixed by #128
Closed

Overloaded function names not displayed as expected #138

dmuhs opened this issue Aug 9, 2023 · 1 comment · Fixed by #128

Comments

@dmuhs
Copy link

dmuhs commented Aug 9, 2023

For example, these two functions have the same name but different parameters:

https://github.com/defi-wonderland/xTokens/blob/dev/solidity/contracts/XERC20Lockbox.sol#L45-L65

The public state-changing methods context view only shows one deposit function, which always indicates the last occurrence.

Similarly, the external calls context only has one deposit function entry. That entry shows the external calls from both functions mixed together.

@vquelque
Copy link

vquelque commented Oct 6, 2023

ack 👍 - will be fixed in the next release.

@vquelque vquelque linked a pull request Oct 13, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants