]> git.lizzy.rs Git - rust.git/commitdiff
std::rt: deny(unused_unsafe)
authorBrian Anderson <banderson@mozilla.com>
Sun, 23 Jun 2013 21:15:39 +0000 (14:15 -0700)
committerBrian Anderson <banderson@mozilla.com>
Tue, 25 Jun 2013 00:07:03 +0000 (17:07 -0700)
src/libstd/rt/mod.rs
src/libstd/rt/test.rs
src/libstd/rt/uv/timer.rs

index fec555e8afd4dfd65603c920b3fabaea803e5315..8713cf05a4772c7dd13b3a5086bd8cc2379c54d9 100644 (file)
@@ -58,6 +58,7 @@
 #[deny(unused_imports)];
 #[deny(unused_mut)];
 #[deny(unused_variable)];
+#[deny(unused_unsafe)];
 
 use cell::Cell;
 use clone::Clone;
@@ -224,7 +225,7 @@ pub fn run(main: ~fn()) -> int {
 
     let nthreads = match os::getenv("RUST_THREADS") {
         Some(nstr) => FromStr::from_str(nstr).get(),
-        None => unsafe { util::num_cpus() }
+        None => util::num_cpus()
     };
 
     // The shared list of sleeping schedulers. Schedulers wake each other
index 36efcd91834b8fd82a613e1d47ebd4e2c3decb70..b0e4968401474750e2c449e22c3877d918e51185 100644 (file)
@@ -74,7 +74,7 @@ pub fn run_in_mt_newsched_task(f: ~fn()) {
     do run_in_bare_thread {
         let nthreads = match os::getenv("RUST_TEST_THREADS") {
             Some(nstr) => FromStr::from_str(nstr).get(),
-            None => unsafe {
+            None => {
                 // Using more threads than cores in test code
                 // to force the OS to preempt them frequently.
                 // Assuming that this help stress test concurrent types.
index cd6fc5c0a250ecd9880136f85a13fe6a619e6d79..14465eb7dfd3afade41774d752c0c5e18ee99955 100644 (file)
@@ -160,14 +160,14 @@ fn repeat_stop() {
                         let mut timer2 = TimerWatcher::new(&mut loop_);
                         do timer2.start(10, 0) |timer2, _| {
 
-                            unsafe { *count_ptr += 1; }
+                            *count_ptr += 1;
 
                             timer2.close(||());
 
                             // Restart the original timer
                             let mut timer = timer;
                             do timer.start(1, 0) |timer, _| {
-                                unsafe { *count_ptr += 1; }
+                                *count_ptr += 1;
                                 timer.close(||());
                             }
                         }