rzerres/conduit
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove pointless double deserialization

Browse Source
This commit is contained in:
Jonas Platte
2021-06-17 20:29:02 +02:00
parent 7fa54e4411
commit af2ce5803e
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/client_server/sync.rs
View File

@@ -286,10 +286,8 @@ pub async fn sync_events_route(
.filter(|(_, pdu)| pdu.kind == EventType::RoomMember) .filter(|(_, pdu)| pdu.kind == EventType::RoomMember)
.map(|(_, pdu)| { .map(|(_, pdu)| {
let content = serde_json::from_value::< let content = serde_json::from_value::<
Raw<ruma::events::room::member::MemberEventContent>, ruma::events::room::member::MemberEventContent,
>(pdu.content.clone()) >(pdu.content.clone())
.expect("Raw::from_value always works")
.deserialize()
.map_err(|_| { .map_err(|_| {
Error::bad_database("Invalid member event in database.") Error::bad_database("Invalid member event in database.")
})?; })?;