Auto-merge pull request #5972 from runrevmark/fix-prop_used_before_assign_error

livecode-vulcan · web-flow · commit 43e34059ff65 · 2017-09-21T13:35:20.000+01:00
[[ Script ]] Fix throwing of 'property used before assigned' error

This patch ensures that the key / value list passed to MCErrorCreateAndThrow
in the PropertyUsedBeforeAssigned throw function is nullptr terminated.
diff --git a/libscript/src/script-error.cpp b/libscript/src/script-error.cpp
@@ -52,7 +52,8 @@ MCScriptThrowPropertyUsedBeforeAssignedError(MCScriptInstanceRef p_instance,
 								 p_instance->module->name,
 								 "property",
 								 MCScriptGetNameOfDefinitionInModule(p_instance->module,
-																		   p_property_def));
+																		   p_property_def),
+                                 nil);
 }
 
 bool

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,8 @@ MCScriptThrowPropertyUsedBeforeAssignedError(MCScriptInstanceRef p_instance,`
`52`	`52`	`p_instance->module->name,`
`53`	`53`	`"property",`
`54`	`54`	`MCScriptGetNameOfDefinitionInModule(p_instance->module,`
`55`		`- p_property_def));`
	`55`	`+ p_property_def),`
	`56`	`+ nil);`
`56`	`57`	`}`
`57`	`58`
`58`	`59`	`bool`