-
Notifications
You must be signed in to change notification settings - Fork 148
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
Return target exec result back to executor #1299
Conversation
@@ -13,7 +13,10 @@ Author: | |||
commy2 | |||
---------------------------------------------------------------------------- */ | |||
|
|||
#define COUNT_WATCH_BOXES 8 | |||
#define COUNT_WATCH_BOXES 8 | |||
#define EXEC_RESULT ([nil] apply {[] call _this} param [0, text ""]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The watch fields report #NIL for nil return value and I like that you can see that something happened instead of void.
Vanilla reports:
nil ->
[nil] -> [<any>]
So it only works for array elements there, but we can be better than that.
I'd wish for text "<any>"
as default value, and also the same for the watch fields (could be part of this PR).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, don't do this multiple spaces inside a line formatting. Should there ever be another macro added with a longer name, all the other spacing will have to be adjusted resulting in a pink wall. Don't do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestions applied.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works.
When merged this pull request will: