@colyseus/schema - npm Package Compare versions

Comparing version 2.0.19 to 2.0.20

lib/changes/ReferenceTracker.d.ts

@@ -11,5 +11,5 @@ import { Ref } from "./ChangeTree";
     addRef(refId: number, ref: Ref, incrementCount?: boolean): void;
-    removeRef(refId: any): void;
+    removeRef(refId: number): void;
     clearRefs(): void;
     garbageCollectDeletedRefs(): void;
 }

lib/changes/ReferenceTracker.js

@@ -28,3 +28,12 @@ "use strict";
     removeRef(refId) {
-        this.refCounts[refId] = this.refCounts[refId] - 1;
+        const refCount = this.refCounts[refId];
+        if (refCount === undefined) {
+            console.warn(`trying to remove reference ${refId} that doesn't exist`);
+            return;
+        }
+        if (refCount === 0) {
+            console.warn(`trying to remove reference ${refId} with 0 refCount`);
+            return;
+        }
+        this.refCounts[refId] = refCount - 1;
         this.deletedRefs.add(refId);
@@ -31,0 +40,0 @@ }

package.json

 {
   "name": "@colyseus/schema",
-  "version": "2.0.19",
+  "version": "2.0.20",
   "description": "Binary state serializer with delta encoding for games",
@@ -5,0 +5,0 @@ "bin": {

src/changes/ReferenceTracker.ts

@@ -30,4 +30,14 @@ import { Schema } from "../Schema";
     // for decoding
-    removeRef(refId) {
-        this.refCounts[refId] = this.refCounts[refId] - 1;
+    removeRef(refId: number) {
+        const refCount = this.refCounts[refId];
+        if (refCount === undefined) {
+            console.warn(`trying to remove reference ${refId} that doesn't exist`);
+            return;
+        }
+        if (refCount === 0) {
+            console.warn(`trying to remove reference ${refId} with 0 refCount`);
+            return;
+        }
+
+        this.refCounts[refId] = refCount - 1;
         this.deletedRefs.add(refId);
@@ -34,0 +44,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1640048

increased by0.22%

Lines of code

20348

increased by0.22%

No dependency changes