@sdeverywhere/compile 0.7.11 → 0.7.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sdeverywhere/compile",
-  "version": "0.7.11",
+  "version": "0.7.12",
   "description": "The core Vensim to C compiler for the SDEverywhere tool suite.",
   "type": "module",
   "main": "./src/index.js",

package/src/_shared/helpers.js

@@ -375,7 +375,3 @@ export let vlog = (title, value, depth = 1) => {
     console.trace()
   }
 }
-export let abend = error => {
-  console.error(error)
-  process.exit(1)
-}

package/src/_shared/read-dat.js

@@ -35,8 +35,16 @@ export async function readDat(pathname, prefix = '') {
     }
   }
 
-  return new Promise(resolve => {
-    let stream = byline(fs.createReadStream(pathname, 'utf8'))
+  return new Promise((resolve, reject) => {
+    // Errors from the read stream aren't propagated by the byline package
+    // so we attach the error handler to `readStream` rather than to `stream`
+    let readStream = fs.createReadStream(pathname, 'utf8')
+    let stream = byline(readStream)
+    readStream.on('error', e => {
+      stream.destroy()
+      reject(new Error(`Failed to read dat file: ${e.message}`))
+    })
+
     stream.on('data', line => {
       let values = splitDatLine(line)
       if (values.length === 1) {
@@ -63,6 +71,7 @@ export async function readDat(pathname, prefix = '') {
       lineNum++
       // if (lineNum % 1e5 === 0) console.log(num(lineNum).format('0,0'))
     })
+
     stream.on('end', () => {
       addValues()
       resolve(log)

package/src/generate/code-gen.js

@@ -1,6 +1,6 @@
 import * as R from 'ramda'
 
-import { asort, lines, strlist, abend, mapIndexed } from '../_shared/helpers.js'
+import { asort, lines, strlist, mapIndexed } from '../_shared/helpers.js'
 import { sub, allDimensions, allMappings, subscriptFamilies } from '../_shared/subscript.js'
 import Model from '../model/model.js'
 
@@ -37,30 +37,26 @@ let codeGenerator = (parsedModel, opts) => {
   function generate() {
     // Read variables and subscript ranges from the model parse tree.
     // This is the main entry point for code generation and is called just once.
-    try {
-      Model.read(parsedModel, spec, extData, directData, modelDirname)
-      // In list mode, print variables to the console instead of generating code.
-      if (operations.includes('printRefIdTest')) {
-        Model.printRefIdTest()
-      }
-      if (operations.includes('printRefGraph')) {
-        Model.printRefGraph(opts.varname)
-      }
-      if (operations.includes('convertNames')) {
-        // Do not generate output, but leave the results of model analysis.
-      }
-      if (operations.includes('generateC')) {
-        // Generate code for each variable in the proper order.
-        let code = emitDeclCode()
-        code += emitInitLookupsCode()
-        code += emitInitConstantsCode()
-        code += emitInitLevelsCode()
-        code += emitEvalCode()
-        code += emitIOCode()
-        return code
-      }
-    } catch (e) {
-      abend(e)
+    Model.read(parsedModel, spec, extData, directData, modelDirname)
+    // In list mode, print variables to the console instead of generating code.
+    if (operations.includes('printRefIdTest')) {
+      Model.printRefIdTest()
+    }
+    if (operations.includes('printRefGraph')) {
+      Model.printRefGraph(opts.varname)
+    }
+    if (operations.includes('convertNames')) {
+      // Do not generate output, but leave the results of model analysis.
+    }
+    if (operations.includes('generateC')) {
+      // Generate code for each variable in the proper order.
+      let code = emitDeclCode()
+      code += emitInitLookupsCode()
+      code += emitInitConstantsCode()
+      code += emitInitLevelsCode()
+      code += emitEvalCode()
+      code += emitIOCode()
+      return code
     }
   }
 

package/src/model/model.js

@@ -454,9 +454,7 @@ function removeUnusedVariables(spec) {
           recordUsedVariable(refVar)
           recordRefsOfVariable(refVar)
         } else {
-          console.error(`No var found for ${refId}`)
-          console.error(v)
-          process.exit(1)
+          throw new Error(`No var found for ${refId} when recording references for ${v.varName}`)
         }
       }
     }
@@ -877,13 +875,7 @@ function sortVarsOfType(varType) {
   // Sort into an lhs dependency list.
   if (PRINT_AUX_GRAPH) printDepsGraph(graph, 'AUX')
   if (PRINT_LEVEL_GRAPH) printDepsGraph(graph, 'LEVEL')
-  let deps
-  try {
-    deps = toposort(graph).reverse()
-  } catch (e) {
-    console.error(e.message)
-    process.exit(1)
-  }
+  let deps = toposort(graph).reverse()
 
   // Turn the dependency-sorted var name list into a var list.
   let sortedVars = varsOfType(
@@ -978,13 +970,7 @@ function sortInitVars() {
   if (PRINT_INIT_GRAPH) printDepsGraph(graph, 'INIT')
 
   // Sort into a reference id dependency list.
-  let deps
-  try {
-    deps = toposort(graph).reverse()
-  } catch (e) {
-    console.error(e.message)
-    process.exit(1)
-  }
+  let deps = toposort(graph).reverse()
 
   // Turn the reference id list into a var list.
   let sortedVars = R.map(refId => varWithRefId(refId), deps)

package/src/model/toposort.js

@@ -42,7 +42,7 @@ function toposort(nodes, edges) {
       } catch (e) {
         nodeRep = ''
       }
-      throw new Error('toposort cyclic dependency:\n' + [...predecessors].join(' →\n') + nodeRep)
+      throw new Error('Found cyclic dependency during toposort:\n' + [...predecessors].join(' →\n') + nodeRep)
     }
 
     if (!nodesHash.has(node)) {