search

idkwim / ouspg

Automatically exported from code.google.com/p/ouspg
3 stars 0 forks source link

Replace setTimeout 0 with process.nextTick in NodeFuzz #79

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I see a number of setTimeout(f, 0) in the source code. This pattern could be 
replaced by the more idiomatic process.nextTick :
http://nodejs.org/api/process.html#process_process_nexttick_callback

Original issue reported on code.google.com by bruan...@gmail.com on 24 Dec 2012 at 12:25

GoogleCodeExporter commented 9 years ago 
The ones I found from nodefuzz.js function sendNewTestCase are now changed to 
nextTick. Thanks for the notification. 

From my point of view the fix is only affecting aesthetics of the code so no 
need for release of a new version.

Original comment by attek...@gmail.com on 24 Dec 2012 at 12:49

GoogleCodeExporter commented 9 years ago 
Oh yeah, totally aesthetics.
It was more of a way to share the existence of process.nextTick than anything 
else ;-)

Original comment by bruan...@gmail.com on 24 Dec 2012 at 12:51

GoogleCodeExporter commented 9 years ago 
Fixed in the NodeFuzz 0.1.1 release.

Original comment by attek...@gmail.com on 8 Jan 2013 at 10:47