diff options
| author | Malo Jaffré <jaffre.malo@gmail.com> | 2017-10-09 20:09:08 +0200 |
|---|---|---|
| committer | Malo Jaffré <jaffre.malo@gmail.com> | 2017-10-09 20:09:08 +0200 |
| commit | 679457ad2a0f33aed6b206107afedcccf5124374 (patch) | |
| tree | c8dadff4dbf6cbf813ced769f221cb6402c6665f /src/libstd/sys_common | |
| parent | eabef0608b030ca8844545837967b29ca4a058b7 (diff) | |
| download | rust-679457ad2a0f33aed6b206107afedcccf5124374.tar.gz rust-679457ad2a0f33aed6b206107afedcccf5124374.zip | |
Refactor to use `debug_struct` in several Debug impls
Fixes #44771.
Diffstat (limited to 'src/libstd/sys_common')
| -rw-r--r-- | src/libstd/sys_common/remutex.rs | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/src/libstd/sys_common/remutex.rs b/src/libstd/sys_common/remutex.rs index 4d0407ccf6c..ce43ec6d9ab 100644 --- a/src/libstd/sys_common/remutex.rs +++ b/src/libstd/sys_common/remutex.rs @@ -116,11 +116,18 @@ impl<T> Drop for ReentrantMutex<T> { impl<T: fmt::Debug + 'static> fmt::Debug for ReentrantMutex<T> { fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { match self.try_lock() { - Ok(guard) => write!(f, "ReentrantMutex {{ data: {:?} }}", &*guard), + Ok(guard) => f.debug_struct("ReentrantMutex").field("data", &*guard).finish(), Err(TryLockError::Poisoned(err)) => { - write!(f, "ReentrantMutex {{ data: Poisoned({:?}) }}", &**err.get_ref()) + f.debug_struct("ReentrantMutex").field("data", &**err.get_ref()).finish() }, - Err(TryLockError::WouldBlock) => write!(f, "ReentrantMutex {{ <locked> }}") + Err(TryLockError::WouldBlock) => { + struct LockedPlaceholder; + impl fmt::Debug for LockedPlaceholder { + fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { f.write_str("<locked>") } + } + + f.debug_struct("ReentrantMutex").field("data", &LockedPlaceholder).finish() + } } } } |
