Skip to content

Commit d71f87d

Browse files
committed
---
yaml --- r: 73295 b: refs/heads/dist-snap c: ee06ed2 h: refs/heads/master i: 73293: 5db4c62 73291: 3349f71 73287: 024fbd7 73279: 4ff2b52 v: v3
1 parent 555e8a5 commit d71f87d

File tree

2 files changed

+23
-14
lines changed

2 files changed

+23
-14
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
77
refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
88
refs/heads/try2: 147ecfdd8221e4a4d4e090486829a06da1e0ca3c
99
refs/heads/incoming: b50030718cf28f2a5a81857a26b57442734fe854
10-
refs/heads/dist-snap: f03c9bd08cde6c83306d91bb07c4810b8b8f13ba
10+
refs/heads/dist-snap: ee06ed2bfd233b57c5989696bb723bddf6569622
1111
refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
1212
refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503
1313
refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0

branches/dist-snap/src/libcore/rt/work_queue.rs

Lines changed: 22 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,39 +11,48 @@
1111
use container::Container;
1212
use option::*;
1313
use vec::OwnedVector;
14+
use unstable::sync::{Exclusive, exclusive};
15+
use cell::Cell;
16+
use kinds::Owned;
1417

1518
pub struct WorkQueue<T> {
16-
priv queue: ~[T]
19+
// XXX: Another mystery bug fixed by boxing this lock
20+
priv queue: ~Exclusive<~[T]>
1721
}
1822

19-
pub impl<T> WorkQueue<T> {
23+
pub impl<T: Owned> WorkQueue<T> {
2024
fn new() -> WorkQueue<T> {
2125
WorkQueue {
22-
queue: ~[]
26+
queue: ~exclusive(~[])
2327
}
2428
}
2529

2630
fn push(&mut self, value: T) {
27-
self.queue.unshift(value)
31+
let value = Cell(value);
32+
self.queue.with(|q| q.unshift(value.take()) );
2833
}
2934

3035
fn pop(&mut self) -> Option<T> {
31-
if !self.queue.is_empty() {
32-
Some(self.queue.shift())
33-
} else {
34-
None
36+
do self.queue.with |q| {
37+
if !q.is_empty() {
38+
Some(q.shift())
39+
} else {
40+
None
41+
}
3542
}
3643
}
3744

3845
fn steal(&mut self) -> Option<T> {
39-
if !self.queue.is_empty() {
40-
Some(self.queue.pop())
41-
} else {
42-
None
46+
do self.queue.with |q| {
47+
if !q.is_empty() {
48+
Some(q.pop())
49+
} else {
50+
None
51+
}
4352
}
4453
}
4554

4655
fn is_empty(&self) -> bool {
47-
return self.queue.is_empty();
56+
self.queue.with_imm(|q| q.is_empty() )
4857
}
4958
}

0 commit comments

Comments
 (0)