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

Add stacktrace functionality and replace (some) uses of Machine.String with it #1812

Closed
thehowl opened this issue Mar 21, 2024 · 0 comments · Fixed by #2145
Closed

Add stacktrace functionality and replace (some) uses of Machine.String with it #1812

thehowl opened this issue Mar 21, 2024 · 0 comments · Fixed by #2145

Comments

@thehowl
Copy link
Member

thehowl commented Mar 21, 2024

          From the review meeting: the agreed solution is to switch from doing `Machine.String()` to printing a stacktrace. (This is likely to be more useful and parseable information by the programmer anyway... :) )

Originally posted by @thehowl in #1736 (comment)

The Machine.String representation is sometimes too verbose or costly to produce to be useful/efficient. A better approach is to print a stacktrace, in a similar fashion to go programs, when for instance panicking within the chain, or within gno test/run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants