When mscript compilation fails, throw error in console.

app/data/cfg.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.8.136",
+  "version": "1.8.138",
   "ext_port": 1111,
   "profiles": {
     "mcopy": {

app/lib/mscript/index.js

@@ -510,8 +510,8 @@ class Mscript {
      * @param line {string} String containing set statement
      */
     set_state(line) {
-        //console.log(`set_state called: ${line}`);
         const update = {};
+        let dist = 0;
         if (line.startsWith('SET CAM2')) {
             update.cam2 = parseInt(line.split('SET CAM2')[1]);
         }
@@ -530,19 +530,14 @@ class Mscript {
         else if (line.startsWith('SET PROJ')) {
             update.proj = parseInt(line.split('SET PROJ')[1]);
         }
-        //console.log(JSON.stringify(update));
         if (this.rec > -1) {
-            for (let key of Object.keys(update)) {
-                this.loops[this.rec][key] = update[key];
+            this.fail(`Line "${line}" is invalid inside of a loop`);
+            return;
         }
-        }
-        else {
         for (let key of Object.keys(update)) {
             this[key] = update[key];
         }
     }
-        console.dir(JSON.stringify(this));
-    }
     /**
      * Return the last loop
      *
@@ -669,7 +664,8 @@ class Mscript {
      */
     update(cmd, val = 1) {
         if (cmd === 'END') {
-            //I don't understand this loop
+            //squashes down loops into the previous one until
+            //the base is reached. val is never not 1, though.
             for (let i = 0; i < val; i++) {
                 if (this.rec === 0) {
                     this.cam += this.loops[this.rec].cam;

app/lib/ui/mscript.js

@@ -140,7 +140,14 @@ PF 1`;
     compile() {
         const data = this.editor.getValue();
         const mscript = new mscript_1.default();
-        const output = mscript.interpret(data);
+        let output = null;
+        try {
+            output = mscript.interpret(data);
+        }
+        catch (err) {
+            mse.console.print(err.toString());
+            return;
+        }
         const len = output.arr.length;
         const cam2 = typeof output.cam2 !== 'undefined' ? `, CAM2 : ${output.cam2}` : '';
         const proj2 = typeof output.proj2 !== 'undefined' ? `, PROJ2 : ${output.proj2}` : '';

app/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "mcopy-app",
-  "version": "1.8.136",
+  "version": "1.8.138",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {

app/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcopy-app",
-  "version": "1.8.136",
+  "version": "1.8.138",
   "description": "GUI for the mcopy small gauge film optical printer platform",
   "main": "main.js",
   "scripts": {

app/src/lib/ui/mscript.ts

@@ -162,7 +162,14 @@ PF 1`;
 	 compile () {
 		const data : string = this.editor.getValue();
 		const mscript : Mscript = new Mscript();
-		const output : any = mscript.interpret(data);
+		let output : any = null;
+		try {
+			output = mscript.interpret(data);
+			this.raw = '';
+		} catch (err) {
+			mse.console.print(err.toString());
+			return;
+		}
 		const len : number = output.arr.length;
 		const cam2 : string = typeof output.cam2 !== 'undefined' ? `, CAM2 : ${output.cam2}` : '';
 		const proj2 : string = typeof output.proj2 !== 'undefined' ? `, PROJ2 : ${output.proj2}` : '';

app/test/mscript.test.js

@@ -126,12 +126,16 @@ LOOP 10
 	PF
 END`;
 
-	it('Should SET state within LOOP', () => {
-		const obj = mscript.interpret(script2)
-		assert.ok(typeof obj === 'object', 'Mscript produced an object response');
-		assert.ok(obj.success, 'Mscript labeled output success');
-		assert.equal(obj.cam, 10, 'Camera state ends at 10');
-		assert.equal(obj.proj, 11, 'Projector state should be 11');
+	it('Should fail when SET state within LOOP', () => {
+		let errorCaught = false;
+		try {
+			const obj = mscript.interpret(script2);
+		} catch (err) {
+			//fail silently
+			errorCaught = true;
+		}
+		assert.ok(typeof obj === 'undefined', 'Mscript fails to produce an output object');
+		assert.ok(errorCaught, 'Error should be thrown by script');
 	});
 });
 

data/cfg.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.8.136",
+  "version": "1.8.138",
   "ext_port": 1111,
   "profiles": {
     "mcopy": {

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "mcopy",
-  "version": "1.8.136",
+  "version": "1.8.138",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "mcopy",
-      "version": "1.8.136",
+      "version": "1.8.138",
       "license": "MIT",
       "dependencies": {
         "alert": "file:app/lib/alert",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcopy",
-  "version": "1.8.136",
+  "version": "1.8.138",
   "description": "Small gauge film optical printer platform",
   "main": "build.js",
   "directories": {

processing/mcopy/cfg.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.8.136",
+  "version": "1.8.138",
   "ext_port": 1111,
   "profiles": {
     "mcopy": {

src/mscript/index.ts

@@ -577,8 +577,8 @@ export default class Mscript {
 	 * @param line {string} String containing set statement
 	 */
 	private set_state (line : string) {
-		//console.log(`set_state called: ${line}`);
 		const update : MscriptUpdatedState = {};
+		let dist : number = 0;
 		if (line.startsWith('SET CAM2')) {
 			update.cam2 = parseInt(line.split('SET CAM2')[1]);
 		} else if (line.startsWith('SET CAMERA2')) {
@@ -592,18 +592,14 @@ export default class Mscript {
 		} else if (line.startsWith('SET PROJ')) {
 			update.proj = parseInt(line.split('SET PROJ')[1]);
 		}
-		//console.log(JSON.stringify(update));
 		if (this.rec > -1) {
-			for (let key of Object.keys(update)) {
-				(this.loops[this.rec] as any)[key] = (update as any)[key];
+			this.fail(`Line "${line}" is invalid inside of a loop`);
+			return;
 		}
-		} else {
 		for (let key of Object.keys(update)) {
 			(this as any)[key] = (update as any)[key];
 		}
 	}
-		console.dir(JSON.stringify(this));
-	}
 	/**
 	 * Return the last loop
 	 *
@@ -730,7 +726,8 @@ export default class Mscript {
 	 */
 	private update (cmd : string, val : number = 1) {
 		if (cmd === 'END') {
-			//I don't understand this loop
+			//squashes down loops into the previous one until
+			//the base is reached. val is never not 1, though.
 			for (let i = 0; i < val; i++) {
 				if (this.rec === 0) {
 					this.cam += this.loops[this.rec].cam;