@bnaya/objectbuffer - npm Package Compare versions

Comparing version 0.0.0-4dc8259 to 0.0.0-50e77f5

dist/internal/saveValue.js

@@ -50,3 +50,2 @@ import { getOurPointerIfApplicable, strByteLength } from "./utils";
         continue;
-        break;
 
@@ -62,3 +61,2 @@ case "string":
         continue;
-        break;
 
@@ -75,3 +73,2 @@ case "bigint":
         continue;
-        break;
 
@@ -82,3 +79,2 @@ case "function":
         continue;
-        break;
 
@@ -89,13 +85,8 @@ case "symbol":
         continue;
-        break;
-      // we will never get here
-
-      case "undefined":
-        continue;
-        break;
-      // we will never get here
-
-      case "boolean":
-        continue;
-        break;
+      // // we will never get here
+      // case "undefined":
+      //   continue;
+      // // we will never get here
+      // case "boolean":
+      //   continue;
     }
@@ -102,0 +93,0 @@

dist/objectbuffer.cjs.js

@@ -574,6 +574,2 @@ Object.defineProperty(exports, "__esModule", {
             continue;
-
-          case "undefined":
-          case "boolean":
-            continue;
         }
@@ -580,0 +576,0 @@ const f = c(n, r.allocator);

dist/objectbuffer.esm.js

@@ -570,6 +570,2 @@ const t = Symbol("INTERNAL_API"), r = 0 + Uint32Array.BYTES_PER_ELEMENT, e = r + Uint32Array.BYTES_PER_ELEMENT, n = e + Uint32Array.BYTES_PER_ELEMENT, i = BigInt("0xFFFFFFFFFFFFFFFF"), o = [ "string", "number", "bigint", "boolean", "undefined" ];
             continue;
-
-          case "undefined":
-          case "boolean":
-            continue;
         }
@@ -576,0 +572,0 @@ const f = c(n, r.allocator);

package.json

 {
   "name": "@bnaya/objectbuffer",
   "description": "Object-like api, backed by an array buffer",
-  "version": "0.0.0-4dc8259",
+  "version": "0.0.0-50e77f5",
   "main": "dist/objectbuffer.cjs.js",
@@ -6,0 +6,0 @@ "module": "dist/index.js",

src/apiTests/bigint.test.ts

@@ -7,4 +7,4 @@ /* eslint-env jest */
 
-describe("object tests", () => {
-  test("Basic object with values", () => {
+describe("bigint tests", () => {
+  test("ok bigint", () => {
     const ob = createObjectBuffer<any>({}, 512, {});
@@ -32,2 +32,26 @@ expect(memoryStats(ob).used).toMatchInlineSnapshot(`160`);
   });
+
+  test("bad bigint", () => {
+    const ob = createObjectBuffer<any>({}, 512, {});
+    expect(memoryStats(ob).used).toMatchInlineSnapshot(`160`);
+
+    ob.bigintNegative = undefined;
+    ob.bigintPositive = undefined;
+
+    expect(memoryStats(ob).used).toMatchInlineSnapshot(`336`);
+
+    ob.bigintNegative = (MAX_64_BIG_INT + BigInt("1")) * BigInt("-1");
+    ob.bigintPositive = MAX_64_BIG_INT + BigInt("1");
+    ob.bigintPositive = undefined;
+    ob.bigintPositive = MAX_64_BIG_INT + BigInt("1");
+
+    expect(memoryStats(ob).used).toMatchInlineSnapshot(`336`);
+
+    expect(ob).toMatchInlineSnapshot(`
+      Object {
+        "bigintNegative": undefined,
+        "bigintPositive": undefined,
+      }
+    `);
+  });
 });

src/internal/saveValue.ts

@@ -91,3 +91,2 @@ import { getOurPointerIfApplicable, strByteLength } from "./utils";
         continue;
-        break;
 
@@ -113,3 +112,2 @@ case "string":
         continue;
-        break;
 
@@ -139,3 +137,2 @@ case "bigint":
         continue;
-        break;
 
@@ -148,3 +145,2 @@ case "function":
         continue;
-        break;
 
@@ -157,12 +153,9 @@ case "symbol":
         continue;
-        break;
 
-      // we will never get here
-      case "undefined":
-        continue;
-        break;
-      // we will never get here
-      case "boolean":
-        continue;
-        break;
+      // // we will never get here
+      // case "undefined":
+      //   continue;
+      // // we will never get here
+      // case "boolean":
+      //   continue;
     }
@@ -169,0 +162,0 @@

No alert changes

Improved metrics

Total package byte prevSize

1319925

increased by0.06%

Number of package files

214

increased by0.47%

Lines of code

16768

increased by0.11%

No dependency changes