diff options
| author | kennytm <kennytm@gmail.com> | 2018-04-17 01:50:59 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-04-17 01:50:59 +0800 |
| commit | e4991b2f48c02d2fc278c5018103530c1e4577e6 (patch) | |
| tree | 596054a639509169c5e37b164a2468e8f90898ab /src/libstd | |
| parent | ccd2c403ac6d1b92c49985d09c4b1d0af8f651e1 (diff) | |
| parent | 4577da75f4badd66a337eb15da76ca836ab1ad6e (diff) | |
| download | rust-e4991b2f48c02d2fc278c5018103530c1e4577e6.tar.gz rust-e4991b2f48c02d2fc278c5018103530c1e4577e6.zip | |
Rollup merge of #49646 - glandium:uninitialized-box, r=alexcrichton
Use box syntax instead of Box::new in Mutex::remutex on Windows The Box::new(mem::uninitialized()) pattern actually actively copies uninitialized bytes from the stack into the box, which is a waste of time. Using the box syntax instead avoids the useless copy.
Diffstat (limited to 'src/libstd')
| -rw-r--r-- | src/libstd/sys/windows/mutex.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libstd/sys/windows/mutex.rs b/src/libstd/sys/windows/mutex.rs index 85560368590..9bf9f749d4d 100644 --- a/src/libstd/sys/windows/mutex.rs +++ b/src/libstd/sys/windows/mutex.rs @@ -117,7 +117,7 @@ impl Mutex { 0 => {} n => return n as *mut _, } - let mut re = Box::new(ReentrantMutex::uninitialized()); + let mut re = box ReentrantMutex::uninitialized(); re.init(); let re = Box::into_raw(re); match self.lock.compare_and_swap(0, re as usize, Ordering::SeqCst) { |
