Skip to content

Commit

Permalink
Fix a bug introduced in previous commit
Browse files Browse the repository at this point in the history
  • Loading branch information
Stjepan Glavina committed Feb 28, 2018
1 parent 082dd6d commit cb56b2d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
4 changes: 2 additions & 2 deletions src/libstd/io/stdio.rs
Expand Up @@ -663,8 +663,8 @@ pub fn set_print(sink: Option<Box<Write + Send>>) -> Option<Box<Write + Send>> {
///
/// This function is used to print error messages, so it takes extra
/// care to avoid causing a panic when `local_stream` is unusable.
/// For instance, if the TLS key for the local stream is uninitialized
/// or already destroyed, or if the local stream is locked by another
/// For instance, if the TLS key for the local stream is
/// already destroyed, or if the local stream is locked by another
/// thread, it will just fall back to the global stream.
///
/// However, if the actual I/O causes an error, this function does panic.
Expand Down
9 changes: 3 additions & 6 deletions src/libstd/thread/local.rs
Expand Up @@ -272,7 +272,7 @@ impl<T: 'static> LocalKey<T> {
///
/// This will lazily initialize the value if this thread has not referenced
/// this key yet. If the key has been destroyed (which may happen if this is called
/// in a destructor), this function will return a ThreadLocalError.
/// in a destructor), this function will return a `ThreadLocalError`.
///
/// # Panics
///
Expand Down Expand Up @@ -484,11 +484,7 @@ mod tests {
assert!(FOO.try_with(|_| ()).is_err());
}
}
fn foo() -> Foo {
assert!(FOO.try_with(|_| ()).is_err());
Foo
}
thread_local!(static FOO: Foo = foo());
thread_local!(static FOO: Foo = Foo);

thread::spawn(|| {
assert!(FOO.try_with(|_| ()).is_ok());
Expand Down Expand Up @@ -520,6 +516,7 @@ mod tests {
impl Drop for S1 {
fn drop(&mut self) {
unsafe {
HITS += 1;
if K2.try_with(|_| ()).is_err() {
assert_eq!(HITS, 3);
} else {
Expand Down

0 comments on commit cb56b2d

Please sign in to comment.