Skip to content

Commit 4b8113f

Browse files
committed
---
yaml --- r: 62551 b: refs/heads/snap-stage3 c: ee06ed2 h: refs/heads/master i: 62549: fe7502b 62547: 47fc83b 62543: 1aaa83b v: v3
1 parent 050ffaa commit 4b8113f

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
@@ -1,7 +1,7 @@
11
---
22
refs/heads/master: 2d28d645422c1617be58c8ca7ad9a457264ca850
33
refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
4-
refs/heads/snap-stage3: f03c9bd08cde6c83306d91bb07c4810b8b8f13ba
4+
refs/heads/snap-stage3: ee06ed2bfd233b57c5989696bb723bddf6569622
55
refs/heads/try: 7b78b52e602bb3ea8174f9b2006bff3315f03ef9
66
refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
77
refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

branches/snap-stage3/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)