Oh, I missed that. So the old code should work unchanged?

On Wed, Apr 3, 2013 at 2:27 PM, Eduard Burtescu <notifications@github.com>wrote:

> In src/node/interpreter.js:
>
> > @@ -38,6 +38,9 @@ function interpret(filename, argv, flags) {
> >    process.argv = ['traceur', filename].concat(argv || []);
> >    process.execArgv = process.execArgv.concat(execArgv);
> >
> > +  if (traceur.options.deferredFunctions)
>
> The nice thing about this is that it should work fine with "vanilla" uses.
> The "vanilla" callback is called before doing anything deferred-specific
> (if it's provided, that is), and 'fs' async methods don't return anything,
> so it's not breaking anything, unless there's some weird code out there
> that depends on 'fs' aync methods to throw an error when they're called
> without a callback or return undefined always.
>
> Because of return value hiding, I disabled setTimeout.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/google/traceur-compiler/pull/237/files#r3642304>
> .
>



-- 
erik