refactor: simplify round function and make it faster

[tarikjabiri]

No description provided.

[tarikjabiri]

@microsoft-github-policy-service agree

[danmarshall]

Hi @tarikjabiri , thanks so much for this valuable contribution! This is definitely one of the most significant bottlenecks in the code. (see #412 ). It is similar to the original code but I didn't use Epsilon for ES5 compatibility.

I'm not sure if people still need support for ES5. I will keep this PR open while I investigate.
I probably should have used a polyfill for epsilon at the time, but Maker.js started with an early TypeScript version and has an antique build system by today's standards.

Note, here is the reference code on MDN from the Wayback machine which has been removed in the current document.

[danmarshall]

Instead of adding a polyfill to Number, I've arrived at defining a scoped variable EPSILON, which can be Number.EPSILON || Math.pow(2, -52). Would you be willing to add this in?

[tarikjabiri]

Thats a good idea, done

[danmarshall]

Perfect, thank you!!