Add tests for basic effects, move handlers to referents for adding and removing.

Moderocky · Moderocky · commit fbdbead2d3e0 · 2022-03-19T11:49:07.000Z
diff --git a/src/main/java/org/byteskript/skript/compiler/FileContext.java b/src/main/java/org/byteskript/skript/compiler/FileContext.java
@@ -91,7 +91,17 @@ public PostCompileClass[] compile() {
             }
         }
         final List<PostCompileClass> classes = new ArrayList<>();
-        classes.add(new PostCompileClass(writer.compile(), writer.getName(), writer.getInternalName()));
+        try {
+            classes.add(new PostCompileClass(writer.compile(), writer.getName(), writer.getInternalName()));
+        } catch (ArrayIndexOutOfBoundsException ex) {
+            if (ex.getStackTrace()[0].getClassName().endsWith("Frame")) {
+                throw new ScriptCompileError(-1, """
+                    Error during assembly phase.
+                    This error cannot be directly triaged, but likely comes from a malformed syntax (in which case the library-maintainer needs to fix it.)
+                    Experienced developers may check the `debug` output to see where the stack calculation error is.
+                    """);
+            }
+        }
         for (ClassBuilder builder : writer.getSuppressed()) {
             try {
                 classes.add(new PostCompileClass(builder.compile(), builder.getName(), builder.getInternalName()));
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/control/AddEffect.java b/src/main/java/org/byteskript/skript/lang/syntax/control/AddEffect.java
@@ -6,7 +6,6 @@
 
 package org.byteskript.skript.lang.syntax.control;
 
-import mx.kenzie.foundation.MethodBuilder;
 import org.byteskript.skript.api.HandlerType;
 import org.byteskript.skript.api.Referent;
 import org.byteskript.skript.api.note.Documentation;
@@ -39,7 +38,7 @@ public AddEffect() {
     
     @Override
     public void preCompile(Context context, Pattern.Match match) throws Throwable {
-        final ElementTree tree = context.getLine();
+        final ElementTree tree = context.getCompileCurrent();
         final ElementTree[] inputs = tree.nested();
         assert inputs.length == 2;
         if (!(inputs[1].current() instanceof final Referent referent))
@@ -49,19 +48,20 @@ public void preCompile(Context context, Pattern.Match match) throws Throwable {
         if (target == null)
             throw new ScriptParseError(context.lineNumber(), "Syntax '" + inputs[1].current()
                 .name() + "' cannot be added to.");
+        inputs[1].type = StandardHandlers.ADD; // todo
         super.preCompile(context, match);
     }
     
     @Override
     public void compile(Context context, Pattern.Match match) throws Throwable {
-        final MethodBuilder method = context.getMethod();
-        assert method != null;
-        final ElementTree tree = context.getLine();
-        final ElementTree[] inputs = tree.nested();
-        final Referent referent = (Referent) inputs[1].current();
-        final Method target = referent.getHandler(StandardHandlers.ADD);
-        assert target != null;
-        this.writeCall(method, target, context);
+//        final MethodBuilder method = context.getMethod();
+//        assert method != null;
+//        final ElementTree tree = context.getCompileCurrent();
+//        final ElementTree[] inputs = tree.nested();
+//        final Referent referent = (Referent) inputs[1].current();
+//        final Method target = referent.getHandler(StandardHandlers.ADD);
+//        assert target != null;
+//        this.writeCall(method, target, context);
         context.setState(CompileState.CODE_BODY);
     }
     
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/control/DeleteEffect.java b/src/main/java/org/byteskript/skript/lang/syntax/control/DeleteEffect.java
@@ -46,7 +46,7 @@ public Pattern.Match match(String thing, Context context) {
     
     @Override
     public void preCompile(Context context, Pattern.Match match) throws Throwable {
-        final ElementTree tree = context.getLine();
+        final ElementTree tree = context.getCompileCurrent();
         final ElementTree[] inputs = tree.nested();
         assert inputs.length == 1;
         inputs[0].type = StandardHandlers.DELETE;
@@ -68,7 +68,7 @@ public void preCompile(Context context, Pattern.Match match) throws Throwable {
     public void compile(Context context, Pattern.Match match) throws Throwable {
         final MethodBuilder method = context.getMethod();
         assert method != null;
-        final ElementTree tree = context.getLine();
+        final ElementTree tree = context.getCompileCurrent();
         final ElementTree[] inputs = tree.nested();
         if (!(inputs[0].current() instanceof VariableExpression)) { // variables have to handle their own deletion
             final Referent referent = (Referent) inputs[0].current();
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/control/RemoveEffect.java b/src/main/java/org/byteskript/skript/lang/syntax/control/RemoveEffect.java
@@ -6,7 +6,6 @@
 
 package org.byteskript.skript.lang.syntax.control;
 
-import mx.kenzie.foundation.MethodBuilder;
 import org.byteskript.skript.api.HandlerType;
 import org.byteskript.skript.api.Referent;
 import org.byteskript.skript.api.note.Documentation;
@@ -39,30 +38,31 @@ public RemoveEffect() {
     
     @Override
     public void preCompile(Context context, Pattern.Match match) throws Throwable {
-        final ElementTree tree = context.getLine();
+        final ElementTree tree = context.getCompileCurrent();
         final ElementTree[] inputs = tree.nested();
         assert inputs.length == 2;
         if (!(inputs[1].current() instanceof final Referent referent))
             throw new ScriptParseError(context.lineNumber(), "Syntax '" + inputs[1].current()
                 .name() + "' cannot be removed from.");
         final Method target = referent.getHandler(StandardHandlers.REMOVE);
         if (target == null)
-            throw new ScriptParseError(context.lineNumber(), "Syntax '" + inputs[0].current()
+            throw new ScriptParseError(context.lineNumber(), "Syntax '" + inputs[1].current()
                 .name() + "' cannot be removed from.");
-        inputs[1].compile = false;
+        inputs[1].type = StandardHandlers.REMOVE; // todo
+//        inputs[1].compile = false;
         super.preCompile(context, match);
     }
     
     @Override
     public void compile(Context context, Pattern.Match match) throws Throwable {
-        final MethodBuilder method = context.getMethod();
-        assert method != null;
-        final ElementTree tree = context.getLine();
-        final ElementTree[] inputs = tree.nested();
-        final Referent referent = (Referent) inputs[1].current();
-        final Method target = referent.getHandler(StandardHandlers.REMOVE);
-        assert target != null;
-        this.writeCall(method, target, context);
+//        final MethodBuilder method = context.getMethod();
+//        assert method != null;
+//        final ElementTree tree = context.getCompileCurrent();
+//        final ElementTree[] inputs = tree.nested();
+//        final Referent referent = (Referent) inputs[1].current();
+//        final Method target = referent.getHandler(StandardHandlers.REMOVE);
+//        assert target != null;
+//        this.writeCall(method, target, context);
         context.setState(CompileState.CODE_BODY);
     }
     
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/variable/AtomicVariableExpression.java b/src/main/java/org/byteskript/skript/lang/syntax/variable/AtomicVariableExpression.java
@@ -39,6 +39,8 @@ public AtomicVariableExpression() {
             handlers.put(StandardHandlers.FIND, AtomicVariable.class.getMethod("get", Object.class));
             handlers.put(StandardHandlers.SET, AtomicVariable.class.getMethod("set", Object.class, Object.class));
             handlers.put(StandardHandlers.DELETE, AtomicVariable.class.getMethod("delete", Object.class));
+            handlers.put(StandardHandlers.ADD, AtomicVariable.class.getMethod("add", Object.class, Object.class));
+            handlers.put(StandardHandlers.REMOVE, AtomicVariable.class.getMethod("remove", Object.class, Object.class));
         } catch (NoSuchMethodException e) {
             e.printStackTrace();
         }
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/variable/GlobalVariableExpression.java b/src/main/java/org/byteskript/skript/lang/syntax/variable/GlobalVariableExpression.java
@@ -42,6 +42,8 @@ public GlobalVariableExpression() {
             handlers.put(StandardHandlers.FIND, GlobalVariableMap.class.getMethod("getVariable", Object.class));
             handlers.put(StandardHandlers.SET, GlobalVariableMap.class.getMethod("setVariable", Object.class, Object.class));
             handlers.put(StandardHandlers.DELETE, GlobalVariableMap.class.getMethod("deleteVariable", Object.class));
+            handlers.put(StandardHandlers.ADD, GlobalVariableMap.class.getMethod("addVariable", Object.class, Object.class));
+            handlers.put(StandardHandlers.REMOVE, GlobalVariableMap.class.getMethod("removeVariable", Object.class, Object.class));
         } catch (NoSuchMethodException e) {
             e.printStackTrace();
         }
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/variable/ThreadVariableExpression.java b/src/main/java/org/byteskript/skript/lang/syntax/variable/ThreadVariableExpression.java
@@ -42,6 +42,8 @@ public ThreadVariableExpression() {
             handlers.put(StandardHandlers.FIND, ThreadVariableMap.class.getMethod("getVariable", Object.class));
             handlers.put(StandardHandlers.SET, ThreadVariableMap.class.getMethod("setVariable", Object.class, Object.class));
             handlers.put(StandardHandlers.DELETE, ThreadVariableMap.class.getMethod("deleteVariable", Object.class));
+            handlers.put(StandardHandlers.ADD, ThreadVariableMap.class.getMethod("addVariable", Object.class, Object.class));
+            handlers.put(StandardHandlers.REMOVE, ThreadVariableMap.class.getMethod("removeVariable", Object.class, Object.class));
         } catch (NoSuchMethodException e) {
             e.printStackTrace();
         }
diff --git a/src/main/java/org/byteskript/skript/lang/syntax/variable/VariableExpression.java b/src/main/java/org/byteskript/skript/lang/syntax/variable/VariableExpression.java
@@ -42,6 +42,7 @@ public VariableExpression() {
         super(SkriptLangSpec.LIBRARY, StandardElements.EXPRESSION, "variable");
         try {
             handlers.put(StandardHandlers.ADD, OperatorHandler.class.getMethod("addObject", Object.class, Object.class));
+            handlers.put(StandardHandlers.REMOVE, OperatorHandler.class.getMethod("removeObject", Object.class, Object.class));
         } catch (NoSuchMethodException e) {
             e.printStackTrace();
         }
@@ -96,6 +97,10 @@ public void compile(Context context, Pattern.Match match) {
         } else if (context.getHandlerMode().equals(StandardHandlers.DELETE)) {
             method.writeCode(WriteInstruction.pushNull());
             method.writeCode(variable.store(slot));
+        } else {
+            method.writeCode(variable.load(slot));
+            this.writeCall(method, handlers.get(context.getHandlerMode()), context);
+            method.writeCode(variable.store(slot));
         }
     }
     
diff --git a/src/main/java/org/byteskript/skript/runtime/internal/GlobalVariableMap.java b/src/main/java/org/byteskript/skript/runtime/internal/GlobalVariableMap.java
@@ -18,6 +18,18 @@
     """)
 public class GlobalVariableMap extends ConcurrentHashMap<String, Object> {
     
+    public static void deleteVariable(Object name) {
+        if (name == null) return;
+        Skript.getVariables().remove(name + "");
+    }
+    
+    public static void addVariable(Object name, Object value) {
+        if (name == null) return;
+        if (value == null) return;
+        final Object original = getVariable(name);
+        setVariable(name, OperatorHandler.addObject(value, original));
+    }
+    
     public static Object getVariable(Object name) {
         if (name == null) return null;
         return Skript.getVariables().get(name + "");
@@ -29,9 +41,11 @@ public static void setVariable(Object name, Object value) {
         else Skript.getVariables().put(name + "", value);
     }
     
-    public static void deleteVariable(Object name) {
+    public static void removeVariable(Object name, Object value) {
         if (name == null) return;
-        Skript.getVariables().remove(name + "");
+        if (value == null) return;
+        final Object original = getVariable(name);
+        setVariable(name, OperatorHandler.subtract(original, value));
     }
     
 }
diff --git a/src/main/java/org/byteskript/skript/runtime/internal/Member.java b/src/main/java/org/byteskript/skript/runtime/internal/Member.java
@@ -181,17 +181,17 @@ public Future<?> run(Skript skript, Object... arguments) {
             private Object value;
             
             @Override
-            public Object result() {
+            public void start() {
+                final Object result = invoker.invoke(arguments);
                 synchronized (this) {
-                    return value;
+                    this.value = result;
                 }
             }
             
             @Override
-            public void start() {
-                final Object result = invoker.invoke(arguments);
+            public Object result() {
                 synchronized (this) {
-                    this.value = result;
+                    return value;
                 }
             }
             
diff --git a/src/main/java/org/byteskript/skript/runtime/internal/OperatorHandler.java b/src/main/java/org/byteskript/skript/runtime/internal/OperatorHandler.java
@@ -75,44 +75,18 @@ public static String concat(String... strings) {
     }
     
     public static Object addObject(Object value, Object to) {
-        if (to instanceof ConfigMap map) ConfigMap.add(value + "", map);
-        else if (to instanceof Collection collection) return collection.add(value);
-        return null;
+        if (to instanceof ConfigMap map) {
+            ConfigMap.add(value + "", map);
+            return map;
+        } else if (to instanceof Collection collection) {
+            collection.add(value);
+            return collection;
+        } else if (to instanceof Object[] array)
+            throw new ScriptRuntimeError("The value '" + value + "' cannot be added to an array.\n"
+                + "\t(Arrays cannot be added to - try using a list instead.)");
+        else return add(to, value);
     }
     
-    public static Iterator<?> acquireIterator(Object thing) {
-        if (thing instanceof Iterable<?> iterable) return iterable.iterator();
-        if (thing instanceof Object[] array) return Arrays.asList(array).iterator();
-        if (thing instanceof Map<?, ?> map) return map.values().iterator();
-        if (thing == null) return Collections.emptyIterator();
-        return Collections.singletonList(thing).iterator();
-    }
-    
-    /**
-     * Moved from {@link org.byteskript.skript.lang.syntax.flow.AssertEffect}
-     */
-    public static void assertion(Object object, Class<?> script, int line) {
-        if (object == null)
-            throw new ScriptAssertionError(script, line);
-        else if (object instanceof Boolean boo && !boo)
-            throw new ScriptAssertionError(script, line);
-        else if (object instanceof Number number && number.intValue() == 0)
-            throw new ScriptAssertionError(script, line);
-    }
-    
-    /**
-     * Moved from {@link org.byteskript.skript.lang.syntax.flow.AssertWithErrorEffect}
-     */
-    public static void assertion(Object object, Object message, Class<?> script, int line) {
-        if (object == null)
-            throw new ScriptAssertionError(script, line, message + "");
-        else if (object instanceof Boolean boo && !boo)
-            throw new ScriptAssertionError(script, line, message + "");
-        else if (object instanceof Number number && number.intValue() == 0)
-            throw new ScriptAssertionError(script, line, message + "");
-    }
-    //endregion
-    
     //region Calculations
     public static Object add(Object a, Object b) {
         if (!(a instanceof Number x) || !(b instanceof Number y)) {
@@ -134,6 +108,19 @@ public static Object add(Object a, Object b) {
         }
     }
     
+    public static Object removeObject(Object value, Object to) {
+        if (to instanceof Map map) {
+            map.remove(value);
+            return map;
+        } else if (to instanceof Collection collection) {
+            collection.remove(value);
+            return collection;
+        } else if (to instanceof Object[] array)
+            throw new ScriptRuntimeError("The value '" + value + "' cannot be added to an array.\n"
+                + "\t(Arrays cannot be added to - try using a list instead.)");
+        else return subtract(to, value);
+    }
+    
     public static Object subtract(Object a, Object b) {
         if (a instanceof String first && b instanceof String second) {
             if (first.endsWith(second)) return first.substring(0, first.length() - second.length());
@@ -158,6 +145,39 @@ public static Object subtract(Object a, Object b) {
         }
     }
     
+    public static Iterator<?> acquireIterator(Object thing) {
+        if (thing instanceof Iterable<?> iterable) return iterable.iterator();
+        if (thing instanceof Object[] array) return Arrays.asList(array).iterator();
+        if (thing instanceof Map<?, ?> map) return map.values().iterator();
+        if (thing == null) return Collections.emptyIterator();
+        return Collections.singletonList(thing).iterator();
+    }
+    //endregion
+    
+    /**
+     * Moved from {@link org.byteskript.skript.lang.syntax.flow.AssertEffect}
+     */
+    public static void assertion(Object object, Class<?> script, int line) {
+        if (object == null)
+            throw new ScriptAssertionError(script, line);
+        else if (object instanceof Boolean boo && !boo)
+            throw new ScriptAssertionError(script, line);
+        else if (object instanceof Number number && number.intValue() == 0)
+            throw new ScriptAssertionError(script, line);
+    }
+    
+    /**
+     * Moved from {@link org.byteskript.skript.lang.syntax.flow.AssertWithErrorEffect}
+     */
+    public static void assertion(Object object, Object message, Class<?> script, int line) {
+        if (object == null)
+            throw new ScriptAssertionError(script, line, message + "");
+        else if (object instanceof Boolean boo && !boo)
+            throw new ScriptAssertionError(script, line, message + "");
+        else if (object instanceof Number number && number.intValue() == 0)
+            throw new ScriptAssertionError(script, line, message + "");
+    }
+    
     public static Object multiply(Object a, Object b) {
         if (a instanceof String string && b instanceof Number number) {
             return string.repeat(Math.max(0, number.intValue()));
diff --git a/src/main/java/org/byteskript/skript/runtime/internal/ThreadVariableMap.java b/src/main/java/org/byteskript/skript/runtime/internal/ThreadVariableMap.java
@@ -15,6 +15,19 @@
 @Ignore
 public class ThreadVariableMap extends HashMap<String, Object> {
     
+    public static void deleteVariable(Object name) {
+        if (name == null) return;
+        if (!(Thread.currentThread() instanceof ScriptThread thread))
+            throw new ScriptRuntimeError("Code is not being run on a script thread - thread variables are unavailable here.");
+        thread.variables.remove(name + "");
+    }
+    
+    public static void addVariable(Object name, Object value) {
+        final Object original = getVariable(name);
+        if (original == null) setVariable(name, value);
+        else setVariable(name, OperatorHandler.add(original, value));
+    }
+    
     public static Object getVariable(Object name) {
         if (name == null) return null;
         if (!(Thread.currentThread() instanceof ScriptThread thread))
@@ -30,11 +43,10 @@ public static void setVariable(Object name, Object value) {
         else thread.variables.put(name + "", value);
     }
     
-    public static void deleteVariable(Object name) {
-        if (name == null) return;
-        if (!(Thread.currentThread() instanceof ScriptThread thread))
-            throw new ScriptRuntimeError("Code is not being run on a script thread - thread variables are unavailable here.");
-        thread.variables.remove(name + "");
+    public static void removeVariable(Object name, Object value) {
+        final Object original = getVariable(name);
+        if (original == null && value instanceof Number) setVariable(name, OperatorHandler.subtract(0, value));
+        else setVariable(name, OperatorHandler.subtract(original, value));
     }
     
 }
diff --git a/src/main/java/org/byteskript/skript/runtime/type/AtomicVariable.java b/src/main/java/org/byteskript/skript/runtime/type/AtomicVariable.java
diff --git a/src/test/resources/tests/addeffect.bsk b/src/test/resources/tests/addeffect.bsk
diff --git a/src/test/resources/tests/deleteeffect.bsk b/src/test/resources/tests/deleteeffect.bsk
diff --git a/src/test/resources/tests/removeeffect.bsk b/src/test/resources/tests/removeeffect.bsk
diff --git a/src/test/resources/tests/seteffect.bsk b/src/test/resources/tests/seteffect.bsk

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,8 @@ public AtomicVariableExpression() {`
`39`	`39`	`handlers.put(StandardHandlers.FIND, AtomicVariable.class.getMethod("get", Object.class));`
`40`	`40`	`handlers.put(StandardHandlers.SET, AtomicVariable.class.getMethod("set", Object.class, Object.class));`
`41`	`41`	`handlers.put(StandardHandlers.DELETE, AtomicVariable.class.getMethod("delete", Object.class));`
	`42`	`+ handlers.put(StandardHandlers.ADD, AtomicVariable.class.getMethod("add", Object.class, Object.class));`
	`43`	`+ handlers.put(StandardHandlers.REMOVE, AtomicVariable.class.getMethod("remove", Object.class, Object.class));`
`42`	`44`	`} catch (NoSuchMethodException e) {`
`43`	`45`	`e.printStackTrace();`
`44`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,8 @@ public GlobalVariableExpression() {`
`42`	`42`	`handlers.put(StandardHandlers.FIND, GlobalVariableMap.class.getMethod("getVariable", Object.class));`
`43`	`43`	`handlers.put(StandardHandlers.SET, GlobalVariableMap.class.getMethod("setVariable", Object.class, Object.class));`
`44`	`44`	`handlers.put(StandardHandlers.DELETE, GlobalVariableMap.class.getMethod("deleteVariable", Object.class));`
	`45`	`+ handlers.put(StandardHandlers.ADD, GlobalVariableMap.class.getMethod("addVariable", Object.class, Object.class));`
	`46`	`+ handlers.put(StandardHandlers.REMOVE, GlobalVariableMap.class.getMethod("removeVariable", Object.class, Object.class));`
`45`	`47`	`} catch (NoSuchMethodException e) {`
`46`	`48`	`e.printStackTrace();`
`47`	`49`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,8 @@ public ThreadVariableExpression() {`
`42`	`42`	`handlers.put(StandardHandlers.FIND, ThreadVariableMap.class.getMethod("getVariable", Object.class));`
`43`	`43`	`handlers.put(StandardHandlers.SET, ThreadVariableMap.class.getMethod("setVariable", Object.class, Object.class));`
`44`	`44`	`handlers.put(StandardHandlers.DELETE, ThreadVariableMap.class.getMethod("deleteVariable", Object.class));`
	`45`	`+ handlers.put(StandardHandlers.ADD, ThreadVariableMap.class.getMethod("addVariable", Object.class, Object.class));`
	`46`	`+ handlers.put(StandardHandlers.REMOVE, ThreadVariableMap.class.getMethod("removeVariable", Object.class, Object.class));`
`45`	`47`	`} catch (NoSuchMethodException e) {`
`46`	`48`	`e.printStackTrace();`
`47`	`49`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,7 @@ public VariableExpression() {`
`42`	`42`	`super(SkriptLangSpec.LIBRARY, StandardElements.EXPRESSION, "variable");`
`43`	`43`	`try {`
`44`	`44`	`handlers.put(StandardHandlers.ADD, OperatorHandler.class.getMethod("addObject", Object.class, Object.class));`
	`45`	`+ handlers.put(StandardHandlers.REMOVE, OperatorHandler.class.getMethod("removeObject", Object.class, Object.class));`
`45`	`46`	`} catch (NoSuchMethodException e) {`
`46`	`47`	`e.printStackTrace();`
`47`	`48`	`}`
`@@ -96,6 +97,10 @@ public void compile(Context context, Pattern.Match match) {`
`96`	`97`	`} else if (context.getHandlerMode().equals(StandardHandlers.DELETE)) {`
`97`	`98`	`method.writeCode(WriteInstruction.pushNull());`
`98`	`99`	`method.writeCode(variable.store(slot));`
	`100`	`+ } else {`
	`101`	`+ method.writeCode(variable.load(slot));`
	`102`	`+ this.writeCall(method, handlers.get(context.getHandlerMode()), context);`
	`103`	`+ method.writeCode(variable.store(slot));`
`99`	`104`	`}`
`100`	`105`	`}`
`101`	`106`
Original file line number	Diff line number	Diff line change
`@@ -181,17 +181,17 @@ public Future<?> run(Skript skript, Object... arguments) {`
`181`	`181`	`private Object value;`
`182`	`182`
`183`	`183`	`@Override`
`184`		`- public Object result() {`
	`184`	`+ public void start() {`
	`185`	`+ final Object result = invoker.invoke(arguments);`
`185`	`186`	`synchronized (this) {`
`186`		`- return value;`
	`187`	`+ this.value = result;`
`187`	`188`	`}`
`188`	`189`	`}`
`189`	`190`
`190`	`191`	`@Override`
`191`		`- public void start() {`
`192`		`- final Object result = invoker.invoke(arguments);`
	`192`	`+ public Object result() {`
`193`	`193`	`synchronized (this) {`
`194`		`- this.value = result;`
	`194`	`+ return value;`
`195`	`195`	`}`
`196`	`196`	`}`
`197`	`197`