Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit f60c89b

Browse files
committed
Applied nitpicks in PromiseQueue shared queue fix.
Set PromiseQueue.prototype._queue to null to avoid confusion. Removed unneeded _curPromise initialization in constructor. Signed-off-by: jayther <jayther@gmail.com>
1 parent 0f0326b commit f60c89b

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

src/utils/Async.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -425,7 +425,6 @@ define(function (require, exports, module) {
425425
*/
426426
function PromiseQueue() {
427427
this._queue = [];
428-
this._curPromise = null;
429428
}
430429

431430
/**
@@ -434,7 +433,7 @@ define(function (require, exports, module) {
434433
* The queue of operations to execute sequentially. Note that even if this array is empty, there might
435434
* still be an operation we need to wait on; that operation's promise is stored in _curPromise.
436435
*/
437-
PromiseQueue.prototype._queue = [];
436+
PromiseQueue.prototype._queue = null;
438437

439438
/**
440439
* @private

0 commit comments

Comments
 (0)