From 6404bef80abcf505983aa7cbc14c3967626734e5 Mon Sep 17 00:00:00 2001 From: Aaron Weiss <aaronweiss74@gmail.com> Date: Sat, 21 Feb 2015 10:31:46 -0500 Subject: [PATCH] More clean-up, and warning fixes. --- src/client/conn.rs | 18 +++++++++--------- src/client/data/message.rs | 4 ++-- src/client/server/mod.rs | 8 ++++---- 3 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/client/conn.rs b/src/client/conn.rs index 09440c8..9ea9ea3 100644 --- a/src/client/conn.rs +++ b/src/client/conn.rs @@ -258,7 +258,7 @@ mod test { Message::new(None, "PRIVMSG", Some(vec!["test"]), Some("Testing!")) ).is_ok()); let data = String::from_utf8(conn.writer().get_ref().to_vec()).unwrap(); - assert_eq!(&data[], "PRIVMSG test :Testing!\r\n"); + assert_eq!(&data[..], "PRIVMSG test :Testing!\r\n"); } #[test] @@ -268,7 +268,7 @@ mod test { let conn = Connection::new(NullReader, MemWriter::new()); assert!(conn.send(exp).is_ok()); let data = String::from_utf8(conn.writer().get_ref().to_vec()).unwrap(); - assert_eq!(&data[], exp); + assert_eq!(&data[..], exp); } #[test] @@ -279,7 +279,7 @@ mod test { Message::new(None, "PRIVMSG", Some(vec!["test"]), Some("€ŠšŽžŒœŸ")), "UTF-8" ).is_ok()); let data = UTF_8.decode(conn.writer().get_ref(), DecoderTrap::Strict).unwrap(); - assert_eq!(&data[], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); + assert_eq!(&data[..], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); } #[test] @@ -289,7 +289,7 @@ mod test { let conn = Connection::new(NullReader, MemWriter::new()); assert!(conn.send(exp, "UTF-8").is_ok()); let data = UTF_8.decode(conn.writer().get_ref(), DecoderTrap::Strict).unwrap(); - assert_eq!(&data[], exp); + assert_eq!(&data[..], exp); } #[test] @@ -300,7 +300,7 @@ mod test { Message::new(None, "PRIVMSG", Some(vec!["test"]), Some("€ŠšŽžŒœŸ")), "l9" ).is_ok()); let data = ISO_8859_15.decode(conn.writer().get_ref(), DecoderTrap::Strict).unwrap(); - assert_eq!(&data[], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); + assert_eq!(&data[..], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); } #[test] @@ -310,7 +310,7 @@ mod test { let conn = Connection::new(NullReader, MemWriter::new()); assert!(conn.send(exp, "l9").is_ok()); let data = ISO_8859_15.decode(conn.writer().get_ref(), DecoderTrap::Strict).unwrap(); - assert_eq!(&data[], exp); + assert_eq!(&data[..], exp); } #[test] @@ -319,7 +319,7 @@ mod test { let conn = Connection::new( MemReader::new("PRIVMSG test :Testing!\r\n".as_bytes().to_vec()), NullWriter ); - assert_eq!(&conn.recv().unwrap()[], "PRIVMSG test :Testing!\r\n"); + assert_eq!(&conn.recv().unwrap()[..], "PRIVMSG test :Testing!\r\n"); } #[test] @@ -328,7 +328,7 @@ mod test { let conn = Connection::new( MemReader::new(b"PRIVMSG test :Testing!\r\n".to_vec()), NullWriter ); - assert_eq!(&conn.recv("UTF-8").unwrap()[], "PRIVMSG test :Testing!\r\n"); + assert_eq!(&conn.recv("UTF-8").unwrap()[..], "PRIVMSG test :Testing!\r\n"); } #[test] @@ -343,6 +343,6 @@ mod test { vec }), NullWriter ); - assert_eq!(&conn.recv("l9").unwrap()[], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); + assert_eq!(&conn.recv("l9").unwrap()[..], "PRIVMSG test :€ŠšŽžŒœŸ\r\n"); } } diff --git a/src/client/data/message.rs b/src/client/data/message.rs index 3744fb7..3bf9935 100644 --- a/src/client/data/message.rs +++ b/src/client/data/message.rs @@ -150,14 +150,14 @@ mod test { args: vec![format!("test")], suffix: Some(format!("Testing!")), }; - assert_eq!(&message.into_string()[], "PRIVMSG test :Testing!\r\n"); + assert_eq!(&message.into_string()[..], "PRIVMSG test :Testing!\r\n"); let message = Message { prefix: Some(format!("test!test@test")), command: format!("PRIVMSG"), args: vec![format!("test")], suffix: Some(format!("Still testing!")), }; - assert_eq!(&message.into_string()[], ":test!test@test PRIVMSG test :Still testing!\r\n"); + assert_eq!(&message.into_string()[..], ":test!test@test PRIVMSG test :Still testing!\r\n"); } #[test] diff --git a/src/client/server/mod.rs b/src/client/server/mod.rs index 31b8a9a..bf4173a 100644 --- a/src/client/server/mod.rs +++ b/src/client/server/mod.rs @@ -144,14 +144,14 @@ impl<T: IrcReader, U: IrcWriter> IrcServer<T, U> { } else if resp == Response::RPL_ENDOFMOTD || resp == Response::ERR_NOMOTD { if self.config.nick_password() != "" { self.send(NICKSERV( - &format!("IDENTIFY {}", self.config.nick_password())[..] + &format!("IDENTIFY {}", self.config.nick_password()) )).unwrap(); } if self.config.umodes() != "" { self.send(MODE(self.config.nickname(), self.config.umodes(), None)).unwrap(); } for chan in self.config.channels().into_iter() { - self.send(JOIN(&chan[..], None)).unwrap(); + self.send(JOIN(&chan, None)).unwrap(); } } else if resp == Response::ERR_NICKNAMEINUSE || resp == Response::ERR_ERRONEOUSNICKNAME { @@ -190,8 +190,8 @@ impl<T: IrcReader, U: IrcWriter> IrcServer<T, U> { } else if let ("MODE", [ref chan, ref mode, ref user]) = (&msg.command[..], &msg.args[..]) { if cfg!(not(feature = "nochanlists")) { if let Some(vec) = self.chanlists.lock().unwrap().get_mut(chan) { - if let Some(n) = vec.as_slice().position_elem(&User::new(&user[..])) { - vec[n].update_access_level(&mode[..]); + if let Some(n) = vec.as_slice().position_elem(&User::new(&user)) { + vec[n].update_access_level(&mode); } } }