---

yaml
---
r: 55527
b: refs/heads/master
c: a2e5827
h: refs/heads/master
i:
  55525: ce13d98
  55523: 919c5e8
  55519: 14651ff
v: v3

Author: brson

[refs]

@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 5f52aecb1e7a53bf46280d057654746c34e89859
+refs/heads/master: a2e5827866876e9bafcf1f4e94f4e354e500420a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 79a2b2eafc3c766cecec8a5f76317693bae9ed17
 refs/heads/try: 8eb2bab100b42f0ba751552d8eff00eb2134c55a

trunk/src/libcore/rt/sched.rs

@@ -60,6 +60,7 @@ impl HackAroundBorrowCk for UnsafeTaskReceiver {
 }
 
 enum CleanupJob {
+    DoNothing,
     RescheduleTask(~Task),
     RecycleTask(~Task),
     GiveTask(~Task, UnsafeTaskReceiver)
@@ -148,6 +149,7 @@ pub impl Scheduler {
 
         // Store the task in the scheduler so it can be grabbed later
         self.current_task = Some(task);
+        self.enqueue_cleanup_job(DoNothing);
 
         // Take pointers to both the task and scheduler's saved registers.
         {
@@ -243,14 +245,13 @@ pub impl Scheduler {
     }
 
     fn run_cleanup_job(&mut self) {
-        rtdebug!("running cleanup jobs");
+        rtdebug!("running cleanup job");
 
-        if self.cleanup_job.is_none() {
-            return;
-        }
+        assert!(self.cleanup_job.is_some());
 
         let cleanup_job = self.cleanup_job.swap_unwrap();
         match cleanup_job {
+            DoNothing => { }
             RescheduleTask(task) => {
                 // NB: Pushing to the *front* of the queue
                 self.task_queue.push_front(task);
@@ -278,9 +279,10 @@ pub impl Scheduler {
             Some(GiveTask(~ref task, _)) => {
                 Some(task)
             }
-            None => {
+            Some(DoNothing) => {
                 None
             }
+            None => fail!(fmt!("all context switches should have a cleanup job"))
         };
         // XXX: Pattern matching mutable pointers above doesn't work
         // because borrowck thinks the three patterns are conflicting