fix: don't create a new promise every time the throttled function is …

…called
jcoreio · May 28, 2023 · 70682fe · 70682fe
1 parent ead88d7
commit 70682fe
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -82,9 +82,10 @@ function throttle<Args: Array<any>, Value>(
     return result
   }
 
-  async function wrapper(...args: Args): Promise<Value> {
+  function wrapper(...args: Args): Promise<Value> {
     nextArgs = nextArgs ? getNextArgs(nextArgs, args) : args
-    if (!nextArgs) throw new Error('unexpected error: nextArgs is null')
+    if (!nextArgs)
+      return Promise.reject(new Error('unexpected error: nextArgs is null'))
     if (nextInvocation) return nextInvocation
     return (nextInvocation = (delay || Promise.resolve()).then(invoke))
   }