[[ Bug 19486 ]] Reset error after failed Java method lookup

runrevmark · runrevmark · commit 1127ea4bc47f · 2017-03-29T15:36:55.000+01:00
This patch ensures that the error generated by a failed method
lookup is reset if binding is not required.
diff --git a/libscript/src/script-instance.cpp b/libscript/src/script-instance.cpp
@@ -835,6 +835,9 @@ __MCScriptResolveForeignFunctionBinding(MCScriptInstanceRef p_instance,
                 return MCScriptThrowUnableToResolveForeignHandlerError(p_instance,
                                                                        p_handler);
             }
+            
+            MCErrorReset();
+            
 			*r_bound = false;
 			
 			return true;

Original file line number	Diff line number	Diff line change
`@@ -835,6 +835,9 @@ __MCScriptResolveForeignFunctionBinding(MCScriptInstanceRef p_instance,`
`835`	`835`	`return MCScriptThrowUnableToResolveForeignHandlerError(p_instance,`
`836`	`836`	`p_handler);`
`837`	`837`	`}`
	`838`	`+`
	`839`	`+ MCErrorReset();`
	`840`	`+`
`838`	`841`	`*r_bound = false;`
`839`	`842`
`840`	`843`	`return true;`