Reduce calls to Thymio.setProgram, fixes #115

Author: sampla

main.qml

@@ -375,18 +375,32 @@ ApplicationWindow {
 
 	Thymio {
 		id: thymio
+		property bool playingOld : false
 		property bool playing: false
-		events: vplEditor ? vplEditor.compiler.output.events : {}
-		source: playing ? vplEditor.compiler.output.script : ""
 		onNodeChanged: playing = false
 		onPlayingChanged: {
-			vplEditor.compiler.execReset(playing);
-			if (!playing) {
-				setVariable("motor.left.target", 0);
-				setVariable("motor.right.target", 0);
-			} else {
-				vplEditor.saveProgram(autosaveName);
+			if (playing && !playingOld) {
+				start();
+			} else if (!playing && playingOld) {
+				stop();
 			}
+			playingOld = playing;
+		}
+		function start() {
+			vplEditor.saveProgram(autosaveName);
+			vplEditor.compiler.execReset(true);
+			var output = vplEditor.compiler.output;
+			program = {
+				events: output.events,
+				source: output.script,
+			};
+		}
+		function stop() {
+			vplEditor.compiler.execReset(false);
+			program = {
+				events: {},
+				source: "",
+			};
 		}
 		onErrorChanged: if (error !== "") { vplEditor.compiler.output.error = error; }
 	}

thymio-vpl2/Thymio.qml

@@ -3,11 +3,6 @@ import Aseba 1.0
 import Simulator 1.0
 
 Item {
-	property var variables: ({})
-	property var events: ({})
-	property string source: ""
-	property string error: ""
-
 	property var node: {
 		for (var i = 0; i < aseba.nodes.length; ++i) {
 			var node = aseba.nodes[i];
@@ -17,19 +12,27 @@ Item {
 		}
 	}
 
+	property var variables: ({})
+	property var program: ({
+		"events": {},
+		"source": "",
+	})
+	property string error: ""
+
 	Simulator {
 		id: simulator
 		onSimulationCompleted: console.log(JSON.stringify(log));
 		onNotify: console.log("Thymio said: " + level + " " + description + " " + arguments)
 	}
 
 	onNodeChanged: {
-		setVariables();
-		setProgram();
+		if (node) {
+			setVariables();
+			setProgram();
+		}
 	}
 	onVariablesChanged: setVariables()
-	onEventsChanged: setProgram()
-	onSourceChanged: setProgram()
+	onProgramChanged: setProgram()
 
 	function setVariables() {
 		if (node) {