Zimm/impress.js — comment on commit eb09f796cbef1f0bbe0b8ef28d814a8c2a43f3e5
File: js/impress.js line 307
By bartaz:

That's really bad. Passing string to `setTimeout` means that `eval` is called on it. And it's not just wrong, but also totally unnecessary in here.

You can simply do:

setTimeout(resetScroll, magicNumber);

BTW, `magicNumber`? :) You could use more understandable variable name ;)

Anyway, as `resetScroll` is only used in this one place it can be even inlined, to make it simpler (and avoid this global you created above):

setTimeout(function () {
sacScroll = true;
}, magicNumber);