]> git.lizzy.rs Git - rust.git/commit
std: Don't deadlock/panic on recursive prints
authorAlex Crichton <alex@alexcrichton.com>
Fri, 27 Mar 2015 23:25:49 +0000 (16:25 -0700)
committerAlex Crichton <alex@alexcrichton.com>
Sat, 28 Mar 2015 02:03:18 +0000 (19:03 -0700)
commite2fd2dffde52a59f7d59d67460aeb2ebf33f77dd
tree1097838bfff9b82a5d1547e4d442316a3b08d747
parent552080181c58beef03493a110b4a38b20b6b5da5
std: Don't deadlock/panic on recursive prints

Previously a panic was generated for recursive prints due to a double-borrow of
a `RefCell`. This was solved by the second borrow's output being directed
towards the global stdout instead of the per-thread stdout (still experimental
functionality).

After this functionality was altered, however, recursive prints still deadlocked
due to the overridden `write_fmt` method which locked itself first and then
wrote all the data. This was fixed by removing the override of the `write_fmt`
method. This means that unlocked usage of `write!` on a `Stdout`/`Stderr` may be
slower due to acquiring more locks, but it's easy to make more performant with a
call to `.lock()`.

Closes #23781
src/libstd/io/stdio.rs
src/libstd/lib.rs
src/test/run-pass/issue-23781.rs [new file with mode: 0644]