Skip to content
This repository was archived by the owner on Apr 20, 2018. It is now read-only.

Commit 4fc39ae

Browse files
Perf improvements for Issue #519
1 parent da39de6 commit 4fc39ae

20 files changed

Lines changed: 54 additions & 40 deletions

dist/rx.all.compat.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6067,8 +6067,8 @@
60676067
if (selector) {
60686068
try {
60696069
results = selector(results);
6070-
} catch (err) {
6071-
return observer.onError(err);
6070+
} catch (e) {
6071+
return observer.onError(e);
60726072
}
60736073

60746074
observer.onNext(results);
@@ -6098,7 +6098,8 @@
60986098
*/
60996099
Observable.fromNodeCallback = function (func, context, selector) {
61006100
return function () {
6101-
for(var args = [], i = 0, len = arguments.length; i < len; i++) { args.push(arguments[i]); }
6101+
var len = arguments.length, args = new Array(len);
6102+
for(var i = 0; i < len; i++) { args[i] = arguments[i]; }
61026103

61036104
return new AnonymousObservable(function (observer) {
61046105
function handler(err) {
@@ -6107,7 +6108,8 @@
61076108
return;
61086109
}
61096110

6110-
for(var results = [], i = 1, len = arguments.length; i < len; i++) { results.push(arguments[i]); }
6111+
var len = arguments.length, results = new Array(len - 1);
6112+
for(var i = 1; i < len; i++) { results[i - 1] = arguments[i]; }
61116113

61126114
if (selector) {
61136115
try {

dist/rx.all.compat.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.all.compat.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.all.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5922,8 +5922,8 @@
59225922
if (selector) {
59235923
try {
59245924
results = selector(results);
5925-
} catch (err) {
5926-
return observer.onError(err);
5925+
} catch (e) {
5926+
return observer.onError(e);
59275927
}
59285928

59295929
observer.onNext(results);
@@ -5953,7 +5953,8 @@
59535953
*/
59545954
Observable.fromNodeCallback = function (func, context, selector) {
59555955
return function () {
5956-
for(var args = [], i = 0, len = arguments.length; i < len; i++) { args.push(arguments[i]); }
5956+
var len = arguments.length, args = new Array(len);
5957+
for(var i = 0; i < len; i++) { args[i] = arguments[i]; }
59575958

59585959
return new AnonymousObservable(function (observer) {
59595960
function handler(err) {
@@ -5962,7 +5963,8 @@
59625963
return;
59635964
}
59645965

5965-
for(var results = [], i = 1, len = arguments.length; i < len; i++) { results.push(arguments[i]); }
5966+
var len = arguments.length, results = new Array(len - 1);
5967+
for(var i = 1; i < len; i++) { results[i - 1] = arguments[i]; }
59665968

59675969
if (selector) {
59685970
try {

dist/rx.all.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.all.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.async.compat.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -309,8 +309,8 @@
309309
if (selector) {
310310
try {
311311
results = selector(results);
312-
} catch (err) {
313-
return observer.onError(err);
312+
} catch (e) {
313+
return observer.onError(e);
314314
}
315315

316316
observer.onNext(results);
@@ -340,7 +340,8 @@
340340
*/
341341
Observable.fromNodeCallback = function (func, context, selector) {
342342
return function () {
343-
for(var args = [], i = 0, len = arguments.length; i < len; i++) { args.push(arguments[i]); }
343+
var len = arguments.length, args = new Array(len);
344+
for(var i = 0; i < len; i++) { args[i] = arguments[i]; }
344345

345346
return new AnonymousObservable(function (observer) {
346347
function handler(err) {
@@ -349,7 +350,8 @@
349350
return;
350351
}
351352

352-
for(var results = [], i = 1, len = arguments.length; i < len; i++) { results.push(arguments[i]); }
353+
var len = arguments.length, results = new Array(len - 1);
354+
for(var i = 1; i < len; i++) { results[i - 1] = arguments[i]; }
353355

354356
if (selector) {
355357
try {

dist/rx.async.compat.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.async.compat.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/rx.async.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -309,8 +309,8 @@
309309
if (selector) {
310310
try {
311311
results = selector(results);
312-
} catch (err) {
313-
return observer.onError(err);
312+
} catch (e) {
313+
return observer.onError(e);
314314
}
315315

316316
observer.onNext(results);
@@ -340,7 +340,8 @@
340340
*/
341341
Observable.fromNodeCallback = function (func, context, selector) {
342342
return function () {
343-
for(var args = [], i = 0, len = arguments.length; i < len; i++) { args.push(arguments[i]); }
343+
var len = arguments.length, args = new Array(len);
344+
for(var i = 0; i < len; i++) { args[i] = arguments[i]; }
344345

345346
return new AnonymousObservable(function (observer) {
346347
function handler(err) {
@@ -349,7 +350,8 @@
349350
return;
350351
}
351352

352-
for(var results = [], i = 1, len = arguments.length; i < len; i++) { results.push(arguments[i]); }
353+
var len = arguments.length, results = new Array(len - 1);
354+
for(var i = 1; i < len; i++) { results[i - 1] = arguments[i]; }
353355

354356
if (selector) {
355357
try {

0 commit comments

Comments
 (0)