Could default exported function perform a tiny bit better?

[realamirhe]

export default function () {
-   var i=0, tmp, x, str='';
+   var tmp, x, len=arguments.length, str='', i=0;
-   while (i < arguments.length) {
+   while (i < len) {
        if (tmp = arguments[i++]) {
            if (x = toVal(tmp)) {
                str && (str += ' ');
                str += x
            }
        }
    }
    return str;
}

Benchmark on only string concatenation shows an increase of about 1,000,000 operation/sec. Note: I got benchmark results only in node, in a really inaccurate way. but theoretically, it must increase the performance.

[lukeed]

Hey, thanks!

This is already done in #26 – I haven't been able to setup a clean bench environment to test this out yet. Caching a foo.length used to not matter, but it may very well matter once again.

It's on my TODO list to go through these PRs.

[hood]

Any news on this?

[lukeed]

This was addressed & merged in the linked PR. Thanks