Merge pull request #2 from cesarParra/perf-improvement

Avoiding re-notifying on set if the value doesn't change.
cesarParra · May 24, 2024 · 1087b4d · 1087b4d
2 parents c2c3e6d + df75fe9
commit 1087b4d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/force-app/lwc/signals/core.js b/force-app/lwc/signals/core.js
@@ -31,6 +31,9 @@ function $signal(value) {
     return _value;
   }
   function setter(newValue) {
+    if (newValue === _value) {
+      return;
+    }
     _value = newValue;
     for (const subscriber of subscribers) {
       subscriber();

diff --git a/src/lwc/signals/core.ts b/src/lwc/signals/core.ts
@@ -52,6 +52,9 @@ function $signal<T>(value: T): Signal<T> {
   }
 
   function setter(newValue: T) {
+    if (newValue === _value) {
+      return;
+    }
     _value = newValue;
     for (const subscriber of subscribers) {
       subscriber();