Use Date.now() instead of +(new Date())

[kurtextrem]

https://jsperf.com/new-date-vs-date-now-vs-performance-now/6

Also saves a few bytes.

Another way would be to just write export default Date.now and then import {now} => import now.

[pablohpsilva]

@kurtextrem I might be wrong, but I think you have changed the now const from a function to a value. I'm not sure where this exported const will be used but I think you should just keep it as a function.

[millette]

export default Date.now

exports the function Date.now() - a value would only be right once after all.

[pablohpsilva]

You are right. My mistake.

[fspoettel]

IMO it would make sense to remove lib/now.mjs at this point since it just re-declares browser functionality. Instead, I'd argue for calling Date.now() directly when needed:

• Importing now() instead of using Date.now() reduces readability of code since I have to look up how now is implemented when reading code calling the func
• A module will lead to some overhead when used with common module bundlers (e.g. webpack), negating the bytes saved with declaring it here

[kurtextrem]

I agree! However, I don't know if there was a plan to add other functions to now.mjs so I kept it.