Fix "process.hrtime() only accepts an Array tuple." error. - #922

Thanks! That's weird – what version of Node is that with? On Node 6, process.hrtime(undefined) works exactly as expected (i.e. the same as process.hrtime().

Rather than redefining toMilliseconds, how about just changing this function to something like this?

time = function time ( previous ) {
    if ( previous ) {
        const hrtime = process.hrtime( previous );
        return hrtime[0] * 1e3 + hrtime[1] / 1e6;
    }

    return process.hrtime();
};

It's Node 4.

It seems rather odd to me to have one function do pretty much unrelated things depending on whether or not you pass it an argument. I'm not sure what to call the split-up time function without sounding redundant with timeStart and timeEnd... I think timeStartHelper and timeEndHelper would be descriptive.

Alright, I split up time using those names.

LGTM, thanks 👍

It seems rather odd to me to have one function do pretty much unrelated things depending on whether or not you pass it an argument

I agree, but the Node.js people apparently don't 😀 I guess we were just following their lead

Released 0.35.7