diff options
author | Manos Pitsidianakis <el13635@mail.ntua.gr> | 2020-02-08 23:18:02 +0200 |
---|---|---|
committer | Manos Pitsidianakis <el13635@mail.ntua.gr> | 2020-02-08 23:56:08 +0200 |
commit | 9b7875c0236f50352f1f495317b2b2ddbdd57553 (patch) | |
tree | 040b058998f728e4155d82cbcb86c33a8cc75a63 /src/components/contacts/contact_list.rs | |
parent | cadb1e1613d59d576bd6e1b3e909b0de831465e5 (diff) |
ui: change Component::get_status return type
There was no reason to return Option<String>, just return String::new()
instead of Option::None
Diffstat (limited to 'src/components/contacts/contact_list.rs')
-rw-r--r-- | src/components/contacts/contact_list.rs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/components/contacts/contact_list.rs b/src/components/contacts/contact_list.rs index 66063796..95e66052 100644 --- a/src/components/contacts/contact_list.rs +++ b/src/components/contacts/contact_list.rs @@ -670,7 +670,7 @@ impl Component for ContactList { context .replies .push_back(UIEvent::StatusEvent(StatusEvent::UpdateStatus( - self.get_status(context).unwrap(), + self.get_status(context), ))); } @@ -707,7 +707,7 @@ impl Component for ContactList { context .replies .push_back(UIEvent::StatusEvent(StatusEvent::UpdateStatus( - self.get_status(context).unwrap(), + self.get_status(context), ))); } return true; @@ -894,10 +894,10 @@ impl Component for ContactList { .unwrap_or(true) } - fn get_status(&self, context: &Context) -> Option<String> { - Some(format!( + fn get_status(&self, context: &Context) -> String { + format!( "{} entries", context.accounts[self.account_pos].address_book.len() - )) + ) } } |