Only call SymInitialize once on Windows

Looks like this can be an expensive operation [1] so we shouldn't be calling
it... once per symbol and stack trace!

[1]: rust-lang/rustup#591

Author: alexcrichton

src/lib.rs

@@ -145,24 +145,15 @@ mod lock {
     }
 }
 
+// requires external synchronization
 #[cfg(all(windows, feature = "dbghelp"))]
-fn dbghelp_init() -> Box<std::any::Any> {
-    use winapi::*;
-    struct Cleanup { handle: HANDLE }
-
-    impl Drop for Cleanup {
-        fn drop(&mut self) {
-            unsafe { ::dbghelp::SymCleanup(self.handle); }
-        }
-    }
-
-    unsafe {
-        let ret = ::dbghelp::SymInitializeW(kernel32::GetCurrentProcess(),
-                                            0 as *mut _, TRUE);
-        if ret != TRUE {
-            Box::new(())
-        } else {
-            Box::new(Cleanup { handle: kernel32::GetCurrentProcess() })
-        }
+unsafe fn dbghelp_init() {
+    static mut INITIALIZED: bool = false;
+
+    if !INITIALIZED {
+        dbghelp::SymInitializeW(kernel32::GetCurrentProcess(),
+                                0 as *mut _,
+                                winapi::TRUE);
+        INITIALIZED = true;
     }
 }