Feature/ls24003545/caller parms

smeup / jariko

a JAva virtual machine Rpg Interpreter written in KOtlin

Apache License 2.0

65 stars 11 forks source link

Feature/ls24003545/caller parms #579

Closed dom-apuliasoft closed 3 months ago

dom-apuliasoft commented 3 months ago

Description

Add a field to keep caller params in InterpreterStatus. This is immediately needed in order to fix the improper evaluation of ParmsExpr.

Related to:

LS24003545

Checklist:

[x] There are tests regarding this feature
[x] The code follows the Kotlin conventions (run ./gradlew ktlintCheck)
[x] The code passes all tests (run ./gradlew check)
[ ] There is a specific documentation in the docs directory

dom-apuliasoft commented 3 months ago

@lanarimarco well done, I missed the suggestion of asserting on the number of passed parameters in your previous comment in a first instance. Very good you noticed it on time.