Run cargo fmt and apply clippy lints.

matrix_sdk_base/src/client.rs

@@ -100,7 +100,7 @@ fn hoist_room_event_prev_content(event: &mut EventJson<RoomEvent>) -> Option<Eve
     let mut ev = event.deserialize().ok()?;
     match &mut ev {
         RoomEvent::RoomMember(ref mut member) if member.prev_content.is_none() => {
-            if let Some(prev) = prev_content.deserialize().ok() {
+            if let Ok(prev) = prev_content.deserialize() {
                 member.prev_content = Some(prev)
             }
 
@@ -123,7 +123,7 @@ fn hoist_state_event_prev_content(event: &EventJson<StateEvent>) -> Option<Event
     let mut ev = event.deserialize().ok()?;
     match &mut ev {
         StateEvent::RoomMember(ref mut member) if member.prev_content.is_none() => {
-            if let Some(prev) = prev_content.deserialize().ok() {
+            if let Ok(prev) = prev_content.deserialize() {
                 member.prev_content = Some(prev)
             }
 

matrix_sdk_base/src/models/room.rs

@@ -589,9 +589,7 @@ impl Room {
         // TODO: This would not be handled correctly as all the MemberEvents have the `prev_content`
         // inside of `unsigned` field.
         match event.membership_change() {
-            Invited | Joined => {
+            Invited | Joined => self.add_member(event),
-                self.add_member(event)
-            }
             Kicked | Banned | KickedAndBanned | InvitationRejected | Left => {
                 self.remove_member(event)
             }