Merge pull request #1232 from srajko/throttle-overflow-fix

johnhaley81 · web-flow · commit 07da8b4b1fcb · 2017-02-28T12:07:31.000-07:00
Use wider int to calculate throttle window
diff --git a/generate/templates/manual/include/callback_wrapper.h b/generate/templates/manual/include/callback_wrapper.h
@@ -57,7 +57,7 @@ class CallbackWrapper {
     }
     // throttle if needed
     uint64_t now = uv_hrtime();
-    if(lastCallTime > 0 && now < lastCallTime + throttle * 1000000) {
+    if(lastCallTime > 0 && now < lastCallTime + throttle * (uint64_t)1000000) {
       // throttled
       return true;
     } else {

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class CallbackWrapper {`
`57`	`57`	`}`
`58`	`58`	`// throttle if needed`
`59`	`59`	`uint64_t now = uv_hrtime();`
`60`		`- if(lastCallTime > 0 && now < lastCallTime + throttle * 1000000) {`
	`60`	`+ if(lastCallTime > 0 && now < lastCallTime + throttle * (uint64_t)1000000) {`
`61`	`61`	`// throttled`
`62`	`62`	`return true;`
`63`	`63`	`} else {`