Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

Script logging should show currently executing script and procedure name #120

Open
darkf opened this issue Dec 28, 2017 · 1 comment
Open

Comments

@darkf
Copy link
Owner

darkf commented Dec 28, 2017

This would make it much easier to track down scripting issues.

@darkf
Copy link
Owner Author

darkf commented Dec 28, 2017

a71e118 makes it so that warnings show the script name, if given by the warn call.

I'd like the currently executing procedure name as well, but it requires either having a separate call stack for procedure names (properly pushed/popped with call/ret opcodes) or setting the current proedure name on call opcodes and reading the address stack for return values and mapping them to procedure names.

If this ends up being too involved, it's fine to drop it and just use the script name.

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

No branches or pull requests

1 participant