Fix build warnings

src/message.rs

@@ -461,7 +461,6 @@ impl Message {
 
 #[cfg(test)]
 mod tests {
-	use ethereum_types::H256;
 	use super::*;
 	use std::time::{self, Duration, SystemTime};
 	use rlp::Rlp;

src/net/mod.rs

@@ -686,26 +686,26 @@ impl<T: MessageHandler> Network<T> {
 }
 
 impl<T: MessageHandler> ::network::NetworkProtocolHandler for Network<T> {
-	fn initialize(&self, io: &NetworkContext) {
+	fn initialize(&self, io: &dyn NetworkContext) {
 		// set up broadcast timer (< 1s)
 		io.register_timer(RALLY_TOKEN, RALLY_TIMEOUT)
 			.expect("Failed to initialize message rally timer");
 	}
 
-	fn read(&self, io: &NetworkContext, peer: &PeerId, packet_id: u8, data: &[u8]) {
+	fn read(&self, io: &dyn NetworkContext, peer: &PeerId, packet_id: u8, data: &[u8]) {
 		self.on_packet(io, peer, packet_id, data)
 	}
 
-	fn connected(&self, io: &NetworkContext, peer: &PeerId) {
+	fn connected(&self, io: &dyn NetworkContext, peer: &PeerId) {
 		// peer with higher ID should begin rallying.
 		self.on_connect(io, peer)
 	}
 
-	fn disconnected(&self, _io: &NetworkContext, peer: &PeerId) {
+	fn disconnected(&self, _io: &dyn NetworkContext, peer: &PeerId) {
 		self.on_disconnect(peer)
 	}
 
-	fn timeout(&self, io: &NetworkContext, timer: TimerToken) {
+	fn timeout(&self, io: &dyn NetworkContext, timer: TimerToken) {
 		// rally with each peer and handle timeouts.
 		match timer {
 			RALLY_TOKEN => self.rally(io),
@@ -719,13 +719,13 @@ impl<T: MessageHandler> ::network::NetworkProtocolHandler for Network<T> {
 pub struct ParityExtensions;
 
 impl ::network::NetworkProtocolHandler for ParityExtensions {
-	fn initialize(&self, _io: &NetworkContext) { }
+	fn initialize(&self, _io: &dyn NetworkContext) { }
 
-	fn read(&self, _io: &NetworkContext, _peer: &PeerId, _id: u8, _msg: &[u8]) { }
+	fn read(&self, _io: &dyn NetworkContext, _peer: &PeerId, _id: u8, _msg: &[u8]) { }
 
-	fn connected(&self, _io: &NetworkContext, _peer: &PeerId) { }
+	fn connected(&self, _io: &dyn NetworkContext, _peer: &PeerId) { }
 
-	fn disconnected(&self, _io: &NetworkContext, _peer: &PeerId) { }
+	fn disconnected(&self, _io: &dyn NetworkContext, _peer: &PeerId) { }
 
-	fn timeout(&self, _io: &NetworkContext, _timer: TimerToken) { }
+	fn timeout(&self, _io: &dyn NetworkContext, _timer: TimerToken) { }
 }

src/rpc/filter.rs

@@ -48,7 +48,7 @@ enum FilterEntry {
 pub struct Manager {
 	key_store: Arc<RwLock<KeyStore>>,
 	filters: RwLock<HashMap<H256, FilterEntry>>,
-	tx: Mutex<mpsc::Sender<Box<Fn() + Send>>>,
+	tx: Mutex<mpsc::Sender<Box<dyn Fn() + Send>>>,
 	join: Option<thread::JoinHandle<()>>,
 	exit: Arc<AtomicBool>,
 }
@@ -57,7 +57,7 @@ impl Manager {
 	/// Create a new filter manager that will dispatch decryption tasks onto
 	/// the given thread pool.
 	pub fn new() -> ::std::io::Result<Self> {
-		let (tx, rx) = mpsc::channel::<Box<Fn() + Send>>();
+		let (tx, rx) = mpsc::channel::<Box<dyn Fn() + Send>>();
 		let exit = Arc::new(AtomicBool::new(false));
 		let e = exit.clone();
 

src/rpc/payload.rs

@@ -66,9 +66,9 @@ fn num_padding_length_bytes(padding_len: usize) -> Option<usize> {
 	let bits = 64 - (padding_len as u64).leading_zeros();
 	match bits {
 		0 => Some(0),
-		0 ... 8 => Some(1),
-		0 ... 16 => Some(2),
-		0 ... 24 => Some(3),
+		0 ..= 8 => Some(1),
+		0 ..= 16 => Some(2),
+		0 ..= 24 => Some(3),
 		_ => None,
 	}
 }