Refactored all types to the new compiled strategy, rewrote tests

Author: WalasPrime

Compiler.js

@@ -8,22 +8,27 @@ module.exports = {
 	 * @param {Number} non_alloc_size If no buffer is provided when encoding then allocate this size. If too small then will throw a runtime exception.
 	 * @returns {Function} (source, buffer=null), where _source_ is the object to be encoded, _buffer_ is the buffer to write to, returns a raw Buffer
 	 */
-	compileEncoder(structure, non_alloc_size=512){
+	compileEncoder(structure, non_alloc_size=4096){
+		let custom_vars = [];
+		const alloc_tmp_var = () => {
+			const name = 'tmp'+custom_vars.length;
+			custom_vars.push(name);
+			return name;
+		}
+		const compiled = structure.compiledEncoder('source', alloc_tmp_var);
 		const code = `
-		(source, buffer=null) => {
-			let position = 0;
+		(source, buffer=null, offset=0) => {
+			let position = offset;
 			let buffer_flexible = false;
 			let i = 0;
 			let tmp;
+			${custom_vars.length > 0 ? 'let '+custom_vars.join(', ')+';' : ''}
 			if(!buffer){
 				buffer = Buffer.alloc(${non_alloc_size});
 				buffer_flexible = true;
 			}
-			${structure.compiledEncoder('source')}
-			if(buffer_flexible){
-				return buffer.slice(0, position);
-			}
-			return buffer;
+			${compiled}
+			return buffer.slice(0, position);
 		}
 		`
 		return eval(code);
@@ -34,12 +39,20 @@ module.exports = {
 	 * @returns {Function} (buffer), where _buffer_ is the data to decode, returns a the decoded _structure_ or throws on errors.
 	 */
 	compileDecoder(structure){
+		let custom_vars = [];
+		const alloc_tmp_var = () => {
+			const name = 'tmp'+custom_vars.length;
+			custom_vars.push(name);
+			return name;
+		}
+		const compiled = structure.compiledDecoder('result', alloc_tmp_var);
 		const code = `
-		(buffer) => {
+		(buffer, offset=0) => {
 			let result;
-			let position = 0;
-			let tmp, tmp2;
-			${structure.compiledDecoder('result')}
+			let position = offset;
+			let tmp;
+			${custom_vars.length > 0 ? 'let '+custom_vars.join(', ')+';' : ''}
+			${compiled}
 			return result;
 		}
 		`

Type.js

@@ -1,11 +1,13 @@
+const { compileEncoder,  compileDecoder } = require('./Compiler');
+
 /**
  * @class
  * Base class for any transcodable type definition.
  */
 class TranscodableType {
 	constructor(){
-		this.last_bytes_encoded = -1;
-		this.last_bytes_decoded = -1;
+		this._tmp_encoder_compiled = null;
+		this._tmp_decoder_compiled = null;
 	}
 	/**
 	 * Encode an object into a binary representation.
@@ -15,15 +17,41 @@ class TranscodableType {
 	 * @returns {Buffer} The buffer that contains the encoded data. If a new buffer is allocated then the data was written from the begining.
 	 */
 	encode(object, buffer, offset){
-		throw Error('Called abstract encode');
+		if(!this._tmp_encoder_compiled)
+			this._tmp_encoder_compiled = compileEncoder(this);
+		return this._tmp_encoder_compiled(object, buffer, offset);
 	}
 	/**
 	 * Decode an object from a binary representation.
 	 * @param {Buffer} buffer The buffer to read from.
 	 * @param {Number} [offset] The offset in buffer to start reading.
 	 */
 	decode(buffer, offset){
-		throw Error('Called abstract decode');
+		if(!this._tmp_decoder_compiled)
+			this._tmp_decoder_compiled = compileDecoder(this);
+		return this._tmp_decoder_compiled(buffer, offset);
+	}
+	/**
+	 * Generate code that encodes the value in variable of name _source\_var_.
+	 * The code is pasted with other generated code in a single function.
+	 * Refer to {@link Compiler.js} for usable local variables.
+	 * @param {String} source_var The variable name to read from.
+	 * @param {function} tmp_var_alloc A function that allocates a unique temporary variable. Returns its name.
+	 * @returns {String} JavaScript code
+	 */
+	compiledEncoder(source_var, tmp_var_alloc){
+		throw new Error('Abstract compiledEncoder called');
+	}
+	/**
+	 * Generate code that decodes the local _buffer_ variable at _position_.
+	 * Save the decoded value to _target\_var_ variable.
+	 * Refer to {@link Compiler.js} for usable local variables.
+	 * @param {String} target_var The variable name to save to.
+	 * @param {function} tmp_var_alloc A function that allocates a unique temporary variable. Returns its name.
+	 * @returns {String} JavaScript code
+	 */
+	compiledDecoder(target_var, tmp_var_alloc){
+		throw new Error('Abstract compiledDecoder called');
 	}
 }