msgpackr 1.9.5-debug.1 → 1.9.6

package/dist/node.cjs

@@ -1324,25 +1324,9 @@ class Packr extends Unpackr {
 				return target.subarray(start, position) // position can change if we call pack again in saveStructures, so we get the buffer now
 			} finally {
 				if (structures) {
-					if (serializationsSinceTransitionRebuild < 10)
-						serializationsSinceTransitionRebuild++;
-					let sharedLength = structures.sharedLength || 0;
-					if (structures.length > sharedLength && !isSequential)
-						structures.length = sharedLength;
-					if (transitionsCount > 10000) {
-						// force a rebuild occasionally after a lot of transitions so it can get cleaned up
-						structures.transitions = null;
-						serializationsSinceTransitionRebuild = 0;
-						transitionsCount = 0;
-						if (recordIdsToRemove.length > 0)
-							recordIdsToRemove = [];
-					} else if (recordIdsToRemove.length > 0 && !isSequential) {
-						for (let i = 0, l = recordIdsToRemove.length; i < l; i++) {
-							recordIdsToRemove[i][RECORD_SYMBOL] = 0;
-						}
-						recordIdsToRemove = [];
-					}
+					resetStructures();
 					if (hasSharedUpdate && packr.saveStructures) {
+						let sharedLength = structures.sharedLength || 0;
 						// we can't rely on start/end with REUSE_BUFFER_MODE since they will (probably) change when we save
 						let returnBuffer = target.subarray(start, position);
 						let newSharedData = prepareStructures$1(structures, packr);
@@ -1358,6 +1342,26 @@ class Packr extends Unpackr {
 					position = start;
 			}
 		};
+		const resetStructures = () => {
+			if (serializationsSinceTransitionRebuild < 10)
+				serializationsSinceTransitionRebuild++;
+			let sharedLength = structures.sharedLength || 0;
+			if (structures.length > sharedLength && !isSequential)
+				structures.length = sharedLength;
+			if (transitionsCount > 10000) {
+				// force a rebuild occasionally after a lot of transitions so it can get cleaned up
+				structures.transitions = null;
+				serializationsSinceTransitionRebuild = 0;
+				transitionsCount = 0;
+				if (recordIdsToRemove.length > 0)
+					recordIdsToRemove = [];
+			} else if (recordIdsToRemove.length > 0 && !isSequential) {
+				for (let i = 0, l = recordIdsToRemove.length; i < l; i++) {
+					recordIdsToRemove[i][RECORD_SYMBOL] = 0;
+				}
+				recordIdsToRemove = [];
+			}
+		};
 		const packArray = (value) => {
 			var length = value.length;
 			if (length < 0x10) {
@@ -1807,8 +1811,9 @@ class Packr extends Unpackr {
 			}
 			// now write the values
 			for (let key in object)
-				if (safePrototype || object.hasOwnProperty(key))
+				if (safePrototype || object.hasOwnProperty(key)) {
 					pack(object[key]);
+				}
 		};
 
 		// craete reference to useRecords if useRecords is a function
@@ -1927,6 +1932,7 @@ class Packr extends Unpackr {
 				position = newPosition;
 				let startTarget = target;
 				pack(value);
+				resetStructures();
 				if (startTarget !== target) {
 					return { position, targetView, target }; // indicate the buffer was re-allocated
 				}
@@ -2870,16 +2876,11 @@ function readStruct(src, position, srcEnd, unpackr) {
 				objectLiteralProperties.push('__proto__:this');
 			}
 			let toObject = (new Function(...args, 'return function(s){return{' + objectLiteralProperties.join(',') + '}}')).apply(null, properties.map(prop => prop.get));
-			try {
-				Object.defineProperty(prototype, 'toJSON', {
-					value(omitUnderscoredProperties) {
-						return toObject.call(this, this[sourceSymbol]);
-					}
-				});
-			} catch(error) {
-				error.message += ' setting properties ' + JSON.stringify(properties);
-				throw error;
-			}
+			Object.defineProperty(prototype, 'toJSON', {
+				value(omitUnderscoredProperties) {
+					return toObject.call(this, this[sourceSymbol]);
+				}
+			});
 		} else {
 			Object.defineProperty(prototype, 'toJSON', {
 				value(omitUnderscoredProperties) {