Exposes bug for rust-lang/rust#18490 in tests/node.rs:46 and

src/node.rs:133

Author: waterlink

src/intercommunication.rs

@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 use std::io::timer::sleep;
 use std::time::duration::Duration;
-use std::comm::{Disconnected, TryRecvError};
+use std::comm::Disconnected;
 
 use std::sync::{Arc, Mutex};
 

src/node.rs

@@ -2,9 +2,7 @@ extern crate time;
 
 use std::io::timer::sleep;
 use std::time::duration::Duration;
-use std::comm::{Disconnected, TryRecvError};
-
-use std::sync::{Arc, Mutex};
+use std::comm::Disconnected;
 
 use std::task::TaskBuilder;
 
@@ -147,7 +145,7 @@ impl Node {
     fn contact(&self) -> &NodeContact {
         match self.contact {
             Some(ref x) => x,
-            None => fail!("You forgot to start the node")
+            None => panic!("You forgot to start the node")
         }
     }
 
@@ -175,7 +173,7 @@ impl NodeService {
     fn start_service(host: String, intercommunication: &mut Intercommunication) -> NodeContact {
         let (contact, service_contact) = NodeService::channels();
 
-        let mut comm = intercommunication.register(host.clone());
+        let comm = intercommunication.register(host.clone());
 
         TaskBuilder::new().named(format!("{}-service", host)).spawn(proc() {
             let mut me = NodeService::new(host, service_contact, comm);

tests/intercommunication.rs

@@ -3,17 +3,14 @@ extern crate raft_rs;
 
 mod using_dumb_network {
 
-    use std::io::timer::sleep;
-    use std::time::duration::Duration;
-
-    use raft_rs::intercommunication::{Intercommunication, DefaultIntercommunication, Package, Ack, Pack, start};
+    use raft_rs::intercommunication::{Intercommunication, DefaultIntercommunication, Ack, Pack, start};
 
     #[test]
     fn sending_simple_ack() {
         let mut comm: DefaultIntercommunication = Intercommunication::new();
 
-        let mut comm_1 = comm.register("host_1".to_string());
-        let mut comm_2 = comm.register("host_2".to_string());
+        let comm_1 = comm.register("host_1".to_string());
+        let comm_2 = comm.register("host_2".to_string());
 
         let stop_comm = start(comm);
 
@@ -24,7 +21,7 @@ mod using_dumb_network {
                 assert_eq!(from, "host_1".to_string());
                 assert_eq!(to, "host_2".to_string());
             },
-            _ => fail!("No ack")
+            _ => panic!("No ack")
         }
 
         stop_comm.send(0);