Expose compiler, use let instead of const, map updates (#118)

* compiler: convert map string values to numeric/bool

* expose CompiledProtodef

* const -> let for some compiler structures

This allows the variables to be updated later on by parameterizable types

* Keep values of mapper type if not matched

If a mapper type isn't mapped, default to the value itself to avoid losing data when reading rather than returning undefined

* document CompiledProtodef

Co-authored-by: Romain Beaumont <romain.rom1@gmail.com>

Author: extremeheat

doc/api.md

@@ -84,6 +84,16 @@ The `path` is an array of namespace keys which select a path of namespaces to be
 
 Compile and return a `ProtoDef` object, optionaly print the generated javascript code.
 
+## CompiledProtodef
+
+The class of which an instance is returned by compileProtoDefSync
+
+It follows the same interface as ProtoDef : read, write, sizeOf, createPacketBuffer, parsePacketBuffer
+Its constructor is CompiledProtodef(sizeOfCtx, writeCtx, readCtx). 
+sizeOfCtx, writeCtx and readCtx are the compiled version of sizeOf, write and read. They are produced by Compiler.compile
+
+It can be used directly for easier debugging/using already compiled js.
+
 ## utils
 
 Some functions that can be useful to build new datatypes reader and writer.

src/compiler.js

@@ -418,5 +418,6 @@ module.exports = {
   ReadCompiler,
   WriteCompiler,
   SizeOfCompiler,
-  ProtoDefCompiler
+  ProtoDefCompiler,
+  CompiledProtodef
 }

src/datatypes/compiler-structures.js

@@ -51,7 +51,7 @@ module.exports = {
           if (name === trueName) names.push(name)
           else names.push(`${name}: ${trueName}`)
         }
-        code += `const { value: ${trueName}, size: ${sizeName} } = ` + compiler.callType(type, offsetExpr) + '\n'
+        code += `let { value: ${trueName}, size: ${sizeName} } = ` + compiler.callType(type, offsetExpr) + '\n'
         offsetExpr += ` + ${sizeName}`
       }
       const sizes = offsetExpr.split(' + ')
@@ -97,7 +97,7 @@ module.exports = {
           trueName = '{' + names.join(', ') + '}'
         } else {
           trueName = compiler.getField(name)
-          code += `const ${trueName} = value.${name}\n`
+          code += `let ${trueName} = value.${name}\n`
         }
         code += 'offset = ' + compiler.callType(trueName, type) + '\n'
       }
@@ -147,7 +147,7 @@ module.exports = {
           trueName = '{' + names.join(', ') + '}'
         } else {
           trueName = compiler.getField(name)
-          code += `const ${trueName} = value.${name}\n`
+          code += `let ${trueName} = value.${name}\n`
         }
         code += 'size += ' + compiler.callType(trueName, type) + '\n'
       }

src/datatypes/compiler-utils.js

@@ -60,7 +60,7 @@ module.exports = {
     }],
     mapper: ['parametrizable', (compiler, mapper) => {
       let code = 'const { value, size } = ' + compiler.callType(mapper.type) + '\n'
-      code += 'return { value: ' + JSON.stringify(sanitizeMappings(mapper.mappings)) + '[value], size }'
+      code += 'return { value: ' + JSON.stringify(sanitizeMappings(mapper.mappings)) + '[value] || value, size }'
       return compiler.wrapCode(code)
     }]
   },
@@ -118,7 +118,7 @@ module.exports = {
     }],
     mapper: ['parametrizable', (compiler, mapper) => {
       const mappings = JSON.stringify(swapMappings(mapper.mappings))
-      const code = 'return ' + compiler.callType(`${mappings}[value]`, mapper.type)
+      const code = 'return ' + compiler.callType(`${mappings}[value] || value`, mapper.type)
       return compiler.wrapCode(code)
     }]
   },
@@ -150,7 +150,7 @@ module.exports = {
     }],
     mapper: ['parametrizable', (compiler, mapper) => {
       const mappings = JSON.stringify(swapMappings(mapper.mappings))
-      const code = 'return ' + compiler.callType(`${mappings}[value]`, mapper.type)
+      const code = 'return ' + compiler.callType(`${mappings}[value] || value`, mapper.type)
       return compiler.wrapCode(code)
     }]
   }
@@ -160,8 +160,11 @@ module.exports = {
 function sanitizeMappings (json) {
   const ret = {}
   for (let key in json) {
-    const val = json[key]
+    let val = json[key]
     key = hex2dec(key)
+    if (!isNaN(val)) val = Number(val)
+    if (val === 'true') val = true
+    if (val === 'false') val = false
     ret[key] = val
   }
   return ret