diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2016-07-16 10:10:17 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-07-16 10:10:17 -0700 |
| commit | d5b9850d79beb87c0e498e0a0b8097e4a11bfae6 (patch) | |
| tree | 3f83945263baa507bcb838e5135fa972e5c1e1d4 | |
| parent | eb196dc2fffe0c992959ed99827654c2adcebe33 (diff) | |
| parent | b28e9dcae8f0f1930bbd11c826dd7357093ea7ae (diff) | |
| download | rust-d5b9850d79beb87c0e498e0a0b8097e4a11bfae6.tar.gz rust-d5b9850d79beb87c0e498e0a0b8097e4a11bfae6.zip | |
Merge pull request #34836 from cynicaldevil/panic-counter
Refactored code to access TLS only in case of panic
| -rw-r--r-- | src/libstd/panicking.rs | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/src/libstd/panicking.rs b/src/libstd/panicking.rs index d73e9542d21..a7de9d3a0c5 100644 --- a/src/libstd/panicking.rs +++ b/src/libstd/panicking.rs @@ -232,10 +232,9 @@ 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 = PANIC_COUNT.with(|s| { - let prev = s.get(); - s.set(0); + let ret; + { let mut to_run = || { slot = Some(f.take().unwrap()()); }; @@ -248,18 +247,21 @@ 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 { - Ok(()) + ret = Ok(()); } else { - Err(mem::transmute(raw::TraitObject { + PANIC_COUNT.with(|s| { + let prev = s.get(); + s.set(prev - 1); + }); + ret = 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() }); |
