clippy

Jenkinsfile

@@ -6,6 +6,12 @@ pipeline {
 	}
 
 	stages {
+		stage('Clippy') {
+			steps {
+				sh "rustup component add clippy"
+				sh "cargo clippy --all-targets --all-features -- -D warnings"
+			}
+		}
 		stage('Build') {
 			steps {
 				sh "cargo install cargo-deb"

src/main.rs

@@ -73,22 +73,19 @@ impl EventEmitter for EventCallback {
 									  room_member: &StrippedStateEventStub<MemberEventContent>,
 									  _: Option<MemberEventContent>) {
 		if room_member.state_key == self.user_id {
-			match sync_room {
-				RoomState::Invited(x) => {
-					let room = x.read().await;
-					println!("Invitation received from {}", room.display_name());
-					match self.client.join_room_by_id(&room.room_id).await {
-						Ok(_) => {
-							println!("Joined {}", room.display_name());
-							self.statsd_client.incr("joined.success");
-						},
-						Err(_) => {
-							println!("There was an error while joining {}", room.display_name());
-							self.statsd_client.incr("joined.error");
-						}
-					};
-				},
-				_ => {}
+			if let RoomState::Invited(x) = sync_room {
+				let room = x.read().await;
+				println!("Invitation received from {}", room.display_name());
+				match self.client.join_room_by_id(&room.room_id).await {
+					Ok(_) => {
+						println!("Joined {}", room.display_name());
+						self.statsd_client.incr("joined.success");
+					},
+					Err(_) => {
+						println!("There was an error while joining {}", room.display_name());
+						self.statsd_client.incr("joined.error");
+					}
+				};
 			}
 		}
 	}

src/relation.rs

@@ -25,13 +25,13 @@ pub async fn react(client: &Client,
 	content_hm.insert("m.relates_to".to_string(),
 					  ContentStruct {
 						  rel_type: "m.annotation".to_string(),
-						  event_id: event_id,
+						  event_id,
 						  key: emoji
 					  });
 	
 	let request = create_message_event::Request {
 		event_type: EventType::Custom("m.reaction".into()),
-		room_id: room_id,
+		room_id,
 		txn_id: Uuid::new_v4().to_string(),
 		data: serde_json::value::to_raw_value(&content_hm).unwrap()
 	};