about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAlex Crichton <alex@alexcrichton.com>2016-07-16 10:10:17 -0700
committerGitHub <noreply@github.com>2016-07-16 10:10:17 -0700
commitd5b9850d79beb87c0e498e0a0b8097e4a11bfae6 (patch)
tree3f83945263baa507bcb838e5135fa972e5c1e1d4
parenteb196dc2fffe0c992959ed99827654c2adcebe33 (diff)
parentb28e9dcae8f0f1930bbd11c826dd7357093ea7ae (diff)
downloadrust-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.rs20
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()
     });