Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

Fix a StreamQueue bug #44

Merged
merged 2 commits into from
Jan 5, 2018
Merged

Fix a StreamQueue bug #44

merged 2 commits into from
Jan 5, 2018

Conversation

nex3
Copy link
Contributor

@nex3 nex3 commented Jan 5, 2018

Previously, StreamQueueTransaction assumed that its request was the
oldest request in the queue at the time at which it was committed or
rejected. This assumption wasn't always correct, and this change
avoids making it.

Previously, StreamQueueTransaction assumed that its request was the
oldest request in the queue at the time at which it was committed or
rejected. This assumption wasn't always correct, and this change
avoids making it.
@nex3 nex3 requested a review from lrhn January 5, 2018 01:08
Copy link
Contributor

@lrhn lrhn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

CHANGELOG.md Outdated

* Fix a bug in `StreamQueue.startTransaction()` and related methods when
rejecting a transaction that isn't the oldest request in the queue.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, makes sense - you can reject a transaction before it gets its first event.

@@ -401,7 +401,7 @@ abstract class StreamQueue<T> {

// ------------------------------------------------------------------
// Methods that may be called from the request implementations to
// control the even stream.
// control the event stream.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@nex3 nex3 merged commit f500b18 into master Jan 5, 2018
@nex3 nex3 deleted the stream-queue-bug branch January 5, 2018 20:50
mosuem pushed a commit to dart-lang/core that referenced this pull request Oct 14, 2024
Previously, StreamQueueTransaction assumed that its request was the
oldest request in the queue at the time at which it was committed or
rejected. This assumption wasn't always correct, and this change
avoids making it.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants