Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(643)

Issue 2540993002: Fix DoubleLinkedListQueue misbehavior on concurrent modification. (Closed)

Created:
4 years ago by Lasse Reichstein Nielsen
Modified:
4 years ago
Reviewers:
floitsch
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix DoubleLinkedListQueue misbehavior on concurrent modification. Documented forEachEntry, firstEntry and lastEntry. Made forEachEntry more useful by making it compatible with removing and adding around the current entry, and documented how it interacts. BUG= http://dartbug.com/27920 R=floitsch@google.com Committed: https://github.com/dart-lang/sdk/commit/9412b54f1fcb24f3cdd7fa9a32833f981cf2ce63

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+72 lines, -15 lines) Patch
M sdk/lib/collection/queue.dart View 7 chunks +71 lines, -14 lines 1 comment Download
M sdk/lib/core/num.dart View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
Lasse Reichstein Nielsen
4 years ago (2016-11-30 08:08:48 UTC) #2
floitsch
LGTM. We got requests from the dart2js team, if these concurrent-modification checks couldn't be assert-mode ...
4 years ago (2016-12-01 18:31:10 UTC) #3
Lasse Reichstein Nielsen
4 years ago (2016-12-06 08:32:10 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
9412b54f1fcb24f3cdd7fa9a32833f981cf2ce63 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698