diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2016-07-16 13:38:06 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-07-16 13:38:06 -0700 |
| commit | 60564d072f058f995258578c93cc059f8f5ac17d (patch) | |
| tree | b409f07a1daf8e2c7aa0e0b62305354e45e7c6d5 | |
| parent | d5b9850d79beb87c0e498e0a0b8097e4a11bfae6 (diff) | |
| parent | dd72785af537c30aaf70f49cf7abdb9700085997 (diff) | |
| download | rust-60564d072f058f995258578c93cc059f8f5ac17d.tar.gz rust-60564d072f058f995258578c93cc059f8f5ac17d.zip | |
Merge pull request #34862 from TimNN/revert-34836-panic-counter
Revert #34836 "Refactored code to access TLS only in case of panic"
| -rw-r--r-- | src/libstd/panicking.rs | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/src/libstd/panicking.rs b/src/libstd/panicking.rs index a7de9d3a0c5..d73e9542d21 100644 --- a/src/libstd/panicking.rs +++ b/src/libstd/panicking.rs @@ -232,9 +232,10 @@ fn default_hook(info: &PanicInfo) { pub unsafe fn try<R, F: FnOnce() -> R>(f: F) -> Result<R, Box<Any + Send>> { let mut slot = None; let mut f = Some(f); - let ret; + let ret = PANIC_COUNT.with(|s| { + let prev = s.get(); + s.set(0); - { let mut to_run = || { slot = Some(f.take().unwrap()()); }; @@ -247,21 +248,18 @@ pub unsafe fn try<R, F: FnOnce() -> R>(f: F) -> Result<R, Box<Any + Send>> { dataptr, &mut any_data, &mut any_vtable); + s.set(prev); + if r == 0 { - ret = Ok(()); + Ok(()) } else { - PANIC_COUNT.with(|s| { - let prev = s.get(); - s.set(prev - 1); - }); - ret = Err(mem::transmute(raw::TraitObject { + Err(mem::transmute(raw::TraitObject { data: any_data as *mut _, vtable: any_vtable as *mut _, - })); + })) } - } + }); - debug_assert!(PANIC_COUNT.with(|c| c.get() == 0)); return ret.map(|()| { slot.take().unwrap() }); |
