]> git.lizzy.rs Git - rust.git/commitdiff
Remove a debug! statement before I/O is ready
authorAlex Crichton <alex@alexcrichton.com>
Sun, 10 Nov 2013 21:05:36 +0000 (13:05 -0800)
committerAlex Crichton <alex@alexcrichton.com>
Sun, 10 Nov 2013 21:05:36 +0000 (13:05 -0800)
The logging macros all use libuv-based I/O, and there was one stray debug
statement in task::spawn which was executing before the I/O context was ready.
Remove it and add a test to make sure that we can continue to debug this sort of
code.

Closes #10405

src/libstd/task/spawn.rs
src/test/run-pass/spawning-with-debug.rs [new file with mode: 0644]

index a4a43a01edd2811038cd9877313f69d1fb6cbdb5..b1d72c063ac8e1aa24e507b7e5dd22df8eba7a35 100644 (file)
@@ -661,7 +661,6 @@ pub fn spawn_raw(mut opts: TaskOpts, f: ~fn()) {
                 };
                 new_sched.bootstrap(bootstrap_task);
 
-                debug!("enqueing join_task");
                 // Now tell the original scheduler to join with this thread
                 // by scheduling a thread-joining task on the original scheduler
                 orig_sched_handle.send(TaskFromFriend(join_task));
diff --git a/src/test/run-pass/spawning-with-debug.rs b/src/test/run-pass/spawning-with-debug.rs
new file mode 100644 (file)
index 0000000..f35c5ef
--- /dev/null
@@ -0,0 +1,22 @@
+// Copyright 2013 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+// exec-env:RUST_LOG=debug
+// xfail-fast
+
+// regression test for issue #10405, make sure we don't call debug! too soon.
+
+use std::task;
+
+fn main() {
+    let mut t = task::task();
+    t.sched_mode(task::SingleThreaded);
+    t.spawn(|| ());
+}