From 5239f2edadcd699a9b84ee5c16f2c60c359a05ab Mon Sep 17 00:00:00 2001 From: Alexander Krotov Date: Sat, 6 Jun 2020 18:29:29 +0300 Subject: [PATCH] dc_receive_imf: replace chat_id.is_error() with chat_id.is_unset() Both methods do the same: compare chat_id to 0. However, in these cases 0 refers to the state when chat_id is not determined yet, because no corresponding chat has been found. All functions that returned 0 to indicate error have already been resultified. --- src/dc_receive_imf.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/dc_receive_imf.rs b/src/dc_receive_imf.rs index eb303cadd..38191ddff 100644 --- a/src/dc_receive_imf.rs +++ b/src/dc_receive_imf.rs @@ -1001,7 +1001,7 @@ async fn create_or_lookup_group( let (mut chat_id, chat_id_verified, _blocked) = chat::get_chat_id_by_grpid(context, &grpid) .await .unwrap_or((ChatId::new(0), false, Blocked::Not)); - if !chat_id.is_error() { + if !chat_id.is_unset() { if chat_id_verified { if let Err(err) = check_verified_properties(context, mime_parser, from_id as u32, to_ids).await @@ -1032,7 +1032,7 @@ async fn create_or_lookup_group( .await .unwrap_or_default(); - if chat_id.is_error() + if chat_id.is_unset() && !mime_parser.is_mailinglist_message() && !grpid.is_empty() && grpname.is_some()