enso-org/enso
1
1
Fork 0
mirror of https://github.com/enso-org/enso.git synced 2024-12-23 22:01:42 +03:00
Code Issues Projects Releases Wiki Activity

fix double-eval of this arguments (#700)

Browse Source
This commit is contained in:
Marcin Kostrzewa 2020-04-29 16:59:07 +02:00 committed by GitHub
parent d03a5a9dde
commit 1254920482
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
engine/runtime/src/main/java/org/enso/interpreter/node/callable/InvokeCallableNode.java
View File

@ -167,6 +167,7 @@ public abstract class InvokeCallableNode extends BaseNode {
Stateful selfResult = thisExecutor.executeThunk((Thunk) selfArgument, state);
selfArgument = selfResult.getValue();
state = selfResult.getState();
arguments[thisArgumentPosition] = selfArgument;
}
Function function = methodResolverNode.execute(symbol, selfArgument);
return this.invokeFunctionNode.execute(function, callerFrame, state, arguments);

11
engine/runtime/src/test/scala/org/enso/interpreter/test/semantic/MethodsTest.scala
View File

@ -13,6 +13,17 @@ class MethodsTest extends InterpreterTest {
eval(code) shouldEqual 11
}
"Method calls" should "execute `this` argument once" in {
val code =
"""
|Unit.foo = 0
|
|main = (IO.println "foo").foo
|""".stripMargin
eval(code)
consumeOut shouldEqual List("foo")
}
"Methods" should "be callable with dot operator" in {
val code =
"""