-
Notifications
You must be signed in to change notification settings - Fork 36
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
New features wish list #15
Comments
Hi @zemse, awesome extension! Would be nice to have an |
Thanks for the great extension! One interesting idea would be to output a mermaid.js sequence diagram of some sort. (maybe reduce the function variables a bit so it's more readable). It would give you a more visual overview of the components and stacktrace. One other feature would be merging proxies and their implementation. Currently they basically print out duplicate traces. |
Thanks! I'll check it out. |
Thanks for the useful extension, it's really helped me get on-top of storage access patterns to reduce unnecessary loads & stores. Although Solidity still adds some unnecessary ones in there for good measure 🤷 It would be really useful to know the source line numbers where SLOAD and SSTORE calls are being triggered from. |
I agree, source lines would be useful. Though I wonder how to fit that in the console display since it already prints a lot of stuff which makes it overwhelming sometimes. Maybe behind a flag/config option. I've created an issue for this. Closing this issue since these pending ones in this issue are in my backlog for a long time and I don't think I can do these anytime soon. But feel free to open a new issue about any feature that can be useful. |
add
,mul
and few more)decode
taskThe text was updated successfully, but these errors were encountered: