Fixed bug in dependency checker that caused it to overwrite variables outside its scope

src/script/context.cpp

@@ -265,12 +265,22 @@ ScriptValueP Context::makeClosure(const ScriptValueP& fun) {
 
 size_t Context::openScope() {
 	level += 1;
+	#ifdef _DEBUG
+		scopes.push_back(shadowed.size());
+		assert(scopes.size() == level);
+	#endif
 	return shadowed.size();
 }
 void Context::closeScope(size_t scope) {
 	assert(level > 0);
 	assert(scope <= shadowed.size());
 	level -= 1;
+	#ifdef _DEBUG
+		assert(!scopes.empty());
+		assert(scopes.back() == scope);
+		scopes.pop_back();
+		assert(scopes.size() == level);
+	#endif
 	// restore shadowed variables
 	while (shadowed.size() > scope) {
 		variables[shadowed.back().variable] = shadowed.back().value;

src/script/context.hpp

@@ -103,6 +103,10 @@ class Context {
 	unsigned int level;
 	/// Stack of values
 	vector<ScriptValueP> stack;
+	#ifdef _DEBUG
+		/// The opened scopes, for sanity checking
+		vector<size_t> scopes;
+	#endif
 	
 	// utility types for dependency analysis
 	struct Jump;

src/script/dependency.cpp

@@ -159,7 +159,7 @@ ScriptValueP Context::dependencies(const Dependency& dep, const Script& script)
 				}
 				// unify bindings
 				FOR_EACH(v, j->bindings) {
-					unify(variables[v.variable].value, v.value.value);
+					setVariable(v.variable, unified(variables[v.variable].value, v.value.value) );
 				}
 				delete j;
 			}
@@ -368,9 +368,8 @@ void Context::getBindings(size_t scope, vector<Binding>& bindings) {
 }
 
 void Context::resetBindings(size_t scope) {
-	// same as closeScope()
-	while (shadowed.size() > scope) {
-		variables[shadowed.back().variable] = shadowed.back().value;
-		shadowed.pop_back();
-	}
+	// close and re-open the scope
+	closeScope(scope);
+	size_t same_scope = openScope();
+	assert(scope == same_scope);
 }