Merge PR #11: Update dependencies and fix some minor details

package.json

@@ -13,11 +13,11 @@
   },
   "license": "MIT",
   "devDependencies": {
-    "chai": "4.3.4",
-    "eslint": "7.32.0",
-    "eslint-config-airbnb-base": "14.2.1",
-    "eslint-plugin-import": "2.25.2",
-    "mocha": "8.2.1"
+    "chai": "4.3.7",
+    "eslint": "8.27.0",
+    "eslint-config-airbnb-base": "15.0.0",
+    "eslint-plugin-import": "2.26.0",
+    "mocha": "10.1.0"
   },
   "scripts": {
     "browserify": "mkdir -p build && npm run core-lib && npm run browserify:main && npm run browserify:som-core && npm run browserify:core-lib-loader",

src/lib/assert.js

@@ -20,8 +20,9 @@
 * THE SOFTWARE.
 */
 // @ts-check
-class AssertionFailedException {
+class AssertionFailedException extends Error {
   constructor() {
+    super();
     // Use V8's native method if available, otherwise fallback
     if ('captureStackTrace' in Error) {
       Error.captureStackTrace(this, AssertionFailedException);

src/som/compiler/MethodGenerationContext.js

@@ -80,16 +80,23 @@ export class MethodGenerationContext {
     }
 
     // return the method - the holder field is to be set later on!
-    return universe.newMethod(this.signature,
+    return universe.newMethod(
+      this.signature,
       this.getSourceSectionForMethod(sourceSection),
-      body, this.locals.length);
+      body,
+      this.locals.length,
+    );
   }
 
   getSourceSectionForMethod(ssBody) {
     return new SourceSection(
       `${this.holderGenc.getName().getString()}>>${this.signature.toString()}`,
-      ssBody.startLine(), ssBody.startColumn(),
-      ssBody.charIndex(), ssBody.length(),
+      ssBody.startLine(),
+
+      ssBody.startColumn(),
+      ssBody.charIndex(),
+
+      ssBody.length(),
     );
   }
 
@@ -179,7 +186,11 @@ export class MethodGenerationContext {
     const self = this.getVariable('self');
     return self.getSuperReadNode(
       this.getOuterSelfContextLevel(),
-      this.holderGenc.getName(), this.holderGenc.isClassSide(), source,
+      this.holderGenc.getName(),
+
+      this.holderGenc.isClassSide(),
+
+      source,
     );
   }
 
@@ -190,29 +201,33 @@ export class MethodGenerationContext {
 
   getLocalWriteNode(variableName, valExpr, source) {
     const variable = this.getVariable(variableName);
-    return variable.getWriteNode(this.getContextLevel(variableName),
-      valExpr, source);
+    return variable.getWriteNode(
+      this.getContextLevel(variableName),
+      valExpr,
+
+      source,
+    );
   }
 
   getNonLocalReturn(expr, source) {
     this.makeCatchNonLocalReturn();
-    return factory.createNonLocalReturn(
-      expr, this.getOuterSelfContextLevel(), source,
-    );
+    return factory.createNonLocalReturn(expr, this.getOuterSelfContextLevel(), source);
   }
 
   getSelfRead(source) {
-    return this.getVariable('self').getReadNode(
-      this.getContextLevel('self'), source,
-    );
+    return this.getVariable('self').getReadNode(this.getContextLevel('self'), source);
   }
 
   getObjectFieldRead(fieldName, source) {
     if (!this.holderGenc.hasField(fieldName)) {
       return null;
     }
-    return factory.createFieldRead(this.getSelfRead(source),
-      this.holderGenc.getFieldIndex(fieldName), source);
+    return factory.createFieldRead(
+      this.getSelfRead(source),
+      this.holderGenc.getFieldIndex(fieldName),
+
+      source,
+    );
   }
 
   getGlobalRead(varName, source) {
@@ -224,8 +239,12 @@ export class MethodGenerationContext {
       return null;
     }
 
-    return factory.createFieldWrite(this.getSelfRead(source), exp,
-      this.holderGenc.getFieldIndex(fieldName), source);
+    return factory.createFieldWrite(
+      this.getSelfRead(source),
+      exp,
+      this.holderGenc.getFieldIndex(fieldName),
+      source,
+    );
   }
 
   /**

src/som/compiler/Parser.js

@@ -255,7 +255,10 @@ export class Parser {
 
   getSource(coord) {
     return new SourceSection(
-      'method', coord.startLine, coord.startColumn, coord.charIndex,
+      'method',
+      coord.startLine,
+      coord.startColumn,
+      coord.charIndex,
       this.lexer.getNumberOfCharactersRead() - coord.charIndex,
     );
   }
@@ -433,9 +436,13 @@ export class Parser {
     const coord = this.getCoordinate();
 
     if (!isIdentifier(this.sym)) {
-      throw new ParseError('Assignments should always target variables or'
+      throw new ParseError(
+        'Assignments should always target variables or'
                 + ' fields, but found instead a %(found)s',
-      Sym.Identifier, this);
+        Sym.Identifier,
+
+        this,
+      );
     }
     const variable = this.assignment();
 
@@ -530,19 +537,15 @@ export class Parser {
   unaryMessage(receiver) {
     const coord = this.getCoordinate();
     const selector = this.unarySelector();
-    return createMessageSend(
-      selector, [receiver], this.getSource(coord),
-    );
+    return createMessageSend(selector, [receiver], this.getSource(coord));
   }
 
   binaryMessage(mgenc, receiver) {
     const coord = this.getCoordinate();
     const msg = this.binarySelector();
     const operand = this.binaryOperand(mgenc);
 
-    return createMessageSend(
-      msg, [receiver, operand], this.getSource(coord),
-    );
+    return createMessageSend(msg, [receiver, operand], this.getSource(coord));
   }
 
   binaryOperand(mgenc) {
@@ -572,9 +575,7 @@ export class Parser {
 
     const msg = universe.symbolFor(kw);
 
-    return createMessageSend(
-      msg, args.slice(), this.getSource(coord),
-    );
+    return createMessageSend(msg, args.slice(), this.getSource(coord));
   }
 
   formula(mgenc) {

src/som/compiler/SourcecodeCompiler.js

@@ -53,9 +53,7 @@ export function compileClassFile(path, file, systemClass, universe) {
     return null;
   }
 
-  const result = compile(
-    new Parser(source, `${path}/${file}.som`), systemClass, universe,
-  );
+  const result = compile(new Parser(source, `${path}/${file}.som`), systemClass, universe);
 
   const cname = result.getName();
   const cnameC = cname.getString();

src/som/compiler/Variable.js

@@ -47,9 +47,7 @@ export class Argument {
   }
 
   getSuperReadNode(contextLevel, holderClass, classSide, source) {
-    return createSuperRead(
-      this, contextLevel, holderClass, classSide, source,
-    );
+    return createSuperRead(this, contextLevel, holderClass, classSide, source);
   }
 
   getIndex() {

src/som/interpreter/MessageSendNode.js

@@ -34,7 +34,8 @@ export class MessageSendNode extends Node {
 
     if (argumentNodes[0].isSuperNode()) {
       this.child_dispatch = this.adopt(new UninitializedSuperDispatchNode(
-        selector, argumentNodes[0].getHolderClass(),
+        selector,
+        argumentNodes[0].getHolderClass(),
         argumentNodes[0].isClassSide(),
       ));
     } else {

src/som/interpreter/NodeFactory.js

@@ -66,9 +66,7 @@ export function createVariableRead(local, contextLevel, source) {
 
 export function createSuperRead(variable, contextLevel, holderClass, classSide, source) {
   assert(holderClass instanceof SSymbol);
-  return new SuperReadNode(
-    holderClass, classSide, contextLevel, variable, source,
-  );
+  return new SuperReadNode(holderClass, classSide, contextLevel, variable, source);
 }
 
 export function createVariableWrite(variable, contextLevel, exp, source) {

src/som/primitives/Primitives.js

@@ -47,19 +47,20 @@ export class Primitives {
     const signature = universe.symbolFor(selector);
 
     // Install the given primitive as an instance primitive in the holder class
-    this.holder.addInstancePrimitive(universe.newPrimitive(
-      signature, primFun, this.holder,
-    ), suppressWarning);
+    this.holder.addInstancePrimitive(
+      universe.newPrimitive(signature, primFun, this.holder),
+      suppressWarning,
+    );
   }
 
   installClassPrimitive(selector, primFun) {
     const signature = universe.symbolFor(selector);
 
     // Install the given primitive as an instance primitive in the class of
     // the holder class
-    this.holder.getClass().addInstancePrimitive(universe.newPrimitive(
-      signature, primFun, this.holder,
-    ));
+    this.holder.getClass().addInstancePrimitive(
+      universe.newPrimitive(signature, primFun, this.holder),
+    );
   }
 
   getEmptyPrimitive(selector) {

src/som/primitives/StringPrimitives.js

@@ -95,7 +95,7 @@ function _isWhiteSpace(_frame, args) {
 function _isLetters(_frame, args) {
   const s = args[0].getEmbeddedString();
 
-  if (RegExp(/^\p{L}+$/, 'u').test(s)) {
+  if (/^\p{L}+$/u.test(s)) {
     return universe.trueObject;
   }
   return universe.falseObject;

src/som/vm/Universe.js

@@ -282,8 +282,12 @@ class Universe {
       const cpEntry = this.classPath[i];
 
       // Load the class from a file and return the loaded class
-      const result = compileClassFile(cpEntry, name.getString(),
-        systemClass, this);
+      const result = compileClassFile(
+        cpEntry,
+        name.getString(),
+        systemClass,
+        this,
+      );
       if (result == null) {
         continue; // continue searching in the class path
       }

src/som/vmobjects/numbers.js

@@ -67,9 +67,7 @@ export class SInteger extends SAbstractObject {
 
   primAdd(right) {
     if (right instanceof SBigInteger) {
-      return intOrBigInt(
-        right.getEmbeddedBigInteger() + BigInt(this.intVal), universe,
-      );
+      return intOrBigInt(right.getEmbeddedBigInteger() + BigInt(this.intVal), universe);
     } if (right instanceof SDouble) {
       return this.toDouble().primAdd(right);
     }
@@ -79,9 +77,7 @@ export class SInteger extends SAbstractObject {
 
   primSubtract(right) {
     if (right instanceof SBigInteger) {
-      return intOrBigInt(
-        BigInt(this.intVal) - right.getEmbeddedBigInteger(), universe,
-      );
+      return intOrBigInt(BigInt(this.intVal) - right.getEmbeddedBigInteger(), universe);
     } if (right instanceof SDouble) {
       return this.toDouble().primSubtract(right);
     }
@@ -91,9 +87,7 @@ export class SInteger extends SAbstractObject {
 
   primMultiply(right) {
     if (right instanceof SBigInteger) {
-      return intOrBigInt(
-        right.getEmbeddedBigInteger().multiply(this.intVal), universe,
-      );
+      return intOrBigInt(right.getEmbeddedBigInteger().multiply(this.intVal), universe);
     } if (right instanceof SDouble) {
       return this.toDouble().primMultiply(right);
     }