@woosh/meep-engine 2.98.1 → 2.98.3

package/build/meep.cjs

@@ -890,6 +890,13 @@ function m4_multiply(out, a, b) {
     out[15] = b0 * a03 + b1 * a13 + b2 * a23 + b3 * a33;
 }
 
+const SignalFlags = {
+    /**
+     * If set - signal will not invoke handlers when dispatched
+     */
+    Silent: 1
+};
+
 /**
  *
  * @enum {number}
@@ -968,18 +975,6 @@ class SignalHandler {
  */
 SignalHandler.prototype.isSignalHandler = true;
 
-const SignalFlags = {
-    /**
-     * If set - signal will not invoke handlers when dispatched
-     */
-    Silent: 1,
-    /**
-     * Is set at the start of the dispatch and cleared at the end
-     * @deprecated
-     */
-    Dispatching: 2
-};
-
 /**
  *
  * @author Alex Goldring
@@ -989,6 +984,7 @@ const SignalFlags = {
 
 /**
  * Common dispatch stack
+ * @readonly
  * @type {SignalHandler[]}
  */
 const dispatch_stack = [];
@@ -1001,40 +997,17 @@ let dispatch_stack_top = 0;
  */
 class Signal {
     /**
-     * Event dispatcher dedicated to single event type
-     * @constructor
-     */
-    constructor() {
-        /**
-         * @private
-         * @type {SignalHandler[]}
-         */
-        this.handlers = [];
-
-        /**
-         * Internal flag bitmask
-         * @private
-         * @type {number}
-         */
-        this.flags = 0;
-    }
-
-    /**
-     * @deprecated
-     * @returns {boolean}
+     * @private
+     * @type {SignalHandler[]}
      */
-    get dispatching() {
-        return this.getFlag(SignalFlags.Dispatching);
-    }
+    handlers = [];
 
     /**
-     * @deprecated
-     * @param {boolean} v
+     * Internal flag bitmask
+     * @private
+     * @type {number}
      */
-    set dispatching(v) {
-        this.writeFlag(SignalFlags.Dispatching, v);
-    }
-
+    flags = 0;
 
     /**
      *
@@ -1096,14 +1069,15 @@ class Signal {
 
 
     /**
-     *
-     * @param {function} h
+     * Checks if a given signal handler is present or not
+     * @param {function} handler
+     * @param {*} [thisArg] if not present, will not be considered
      * @returns {boolean}
      */
-    contains(h) {
+    contains(handler, thisArg) {
         const handlers = this.handlers;
 
-        const i = findSignalHandlerIndexByHandle(handlers, h);
+        const i = findSignalHandlerIndexByHandle(handlers, handler, thisArg);
 
         return i !== -1;
     }
@@ -1125,7 +1099,7 @@ class Signal {
     }
 
     /**
-     *
+     * Handler will only be run once, it will be removed automatically after the first execution
      * @param {function} h
      * @param {*} [context]
      */
@@ -1172,7 +1146,10 @@ class Signal {
     }
 
     /**
-     * Remove all handlers
+     * Remove all handlers.
+     * Please note that this will remove even all handlers, irrespective of where they were added from. If another script is attaching to the same signal, using this method will potentially break that code.
+     * For most use cases, prefer to use {@link remove} method instead, or make use of {@link SignalBinding} if you need to keep track of multiple handlers
+     * NOTE: Consider this method to be unsafe, only use it when you understand the implications
      */
     removeAll() {
         const handlers = this.handlers;
@@ -1180,6 +1157,7 @@ class Signal {
     }
 
     /**
+     * NOTE: because of polymorphic call-site nature of this method, it's always better for performance to use monomorphic methods like `send0`, `send1` etc.
      * @param {...*} args
      */
     dispatch(...args) {
@@ -1188,13 +1166,8 @@ class Signal {
             return;
         }
 
-        //mark dispatch process
-        this.setFlag(SignalFlags.Dispatching);
-
         dispatchViaProxy(this.handlers, args);
 
-        //mark end of dispatch process
-        this.clearFlag(SignalFlags.Dispatching);
     }
 
     /**
@@ -1560,14 +1533,6 @@ class Signal {
         dispatch_stack_top = stack_pointer;
     }
 
-    /**
-     * @deprecated do not use
-     * @returns {boolean}
-     */
-    isDispatching() {
-        return this.getFlag(SignalFlags.Dispatching);
-    }
-
     /**
      *
      * @param {Signal} other
@@ -1598,7 +1563,7 @@ Signal.prototype.isSignal = true;
  *
  * @param {SignalHandler[]} handlers
  * @param {function} f
- * @param thisArg
+ * @param {*} [thisArg]
  * @returns {number} index of the handler, or -1 if not found
  */
 function findSignalHandlerIndexByHandle(handlers, f, thisArg) {
@@ -68276,6 +68241,23 @@ class SignalBinding {
         this.#linked = false;
         this.signal.remove(this.handler, this.context);
     }
+
+    /**
+     * Creates a {@link SignalBinding} and links it immediately. Utility method
+     * @param {Signal} signal
+     * @param {function} handler
+     * @param {*} [context]
+     * @return {SignalBinding}
+     */
+    static bind(
+        signal, handler, context
+    ) {
+        const r = new SignalBinding(signal, handler, context);
+
+        r.link();
+
+        return r;
+    }
 }
 
 /**