fix(dispatch): remove infinite loop limit

Author: FabienMotte

README.md

@@ -146,12 +146,6 @@ mySignal.add(listener)
 mySignal.removeAll()
 ```
 
-## Notes
-
-In order to prevent an __infinite loop__ if a signal is dispatching itself in an added listener, the maximum dispatch limit is fixed to __512__.
-
-If this limit is reached, it throws an error.
-
 ## API
 
 See [https://fm-ph.github.io/quark-signal/](https://fm-ph.github.io/quark-signal/)

src/index.js

@@ -1,10 +1,3 @@
-/**
- * @constant
- * @type number
- * @default
- */
-const MAX_DISPATCH_NB = 512
-
 /**
  * Signal class
  *
@@ -143,10 +136,6 @@ class Signal {
   dispatch (...args) {
     this._dispatchNb++
 
-    if (this._dispatchNb > MAX_DISPATCH_NB) {
-      throw new Error('Signal.dispatch() : Maximum dispatch limit reached (prevent infinite loop)')
-    }
-
     for (let i = 0; i < this._listeners.length; i++) {
       const listener = this._listeners[i]
 

test/index.js

@@ -166,16 +166,6 @@ test.cb('create a signal and use a custom listener context', t => {
   t.context.signal.dispatch()
 })
 
-test('create a signal that dispatch itself throws an error', t => {
-  t.context.signal.add(() => {
-    t.context.signal.dispatch()
-  })
-
-  const error = t.throws(() => t.context.signal.dispatch(), Error)
-
-  t.is(error.message, 'Signal.dispatch() : Maximum dispatch limit reached (prevent infinite loop)')
-})
-
 test('create a signal with a listener that stop the propagation', t => {
   t.context.signal.add(() => false)
   t.context.signal.add(() => { })