GDScript: Don't use the NIL address to hold return value of functions #71107
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This prevents that the NIL address is filled with another value, which causes problems for some instructions that read from NIL.
This is still a bit of a flimsy solution, we need to find a way to disambiguate the
NIL
target (i.e. discard the return value) and theNIL
source (i.e. use a Variant of typeNIL
). I thought of some ways to do it but any of those would take a while to implement and test, so it's best if done later.Fix #70964
Fix #70936
Fix #70953