Use `if let` more

src/api/core/accounts.rs

@@ -139,10 +139,8 @@ fn register(data: JsonUpcase<RegisterData>, conn: DbConn) -> EmptyResult {
             }
 
             user.last_verifying_at = Some(user.created_at);
-        } else {
+        } else if let Err(e) = mail::send_welcome(&user.email) {
-            if let Err(e) = mail::send_welcome(&user.email) {
+            error!("Error sending welcome email: {:#?}", e);
-                error!("Error sending welcome email: {:#?}", e);
-            }
         }
     }
 

src/api/icons.rs

@@ -352,10 +352,12 @@ fn get_favicons_node(node: &std::rc::Rc<markup5ever_rcdom::Node>, icons: &mut Ve
                 }
             }
 
-            if has_rel && href.is_some() {
+            if has_rel {
-                if let Ok(full_href) = url.join(&href.unwrap()).map(|h| h.into_string()) {
+                if let Some(inner_href) = href {
-                    let priority = get_icon_priority(&full_href, sizes);
+                    if let Ok(full_href) = url.join(&inner_href).map(|h| h.into_string()) {
-                    icons.push(Icon::new(priority, full_href));
+                        let priority = get_icon_priority(&full_href, sizes);
+                        icons.push(Icon::new(priority, full_href));
+                    }
                 }
             }
         }

src/api/notifications.rs

@@ -166,8 +166,8 @@ impl WSHandler {
         if let Some(params) = path.split('?').nth(1) {
             let params_iter = params.split('&').take(1);
             for val in params_iter {
-                if val.starts_with(ACCESS_TOKEN_KEY) {
+                if let Some(stripped) = val.strip_prefix(ACCESS_TOKEN_KEY) {
-                    return Some(val[ACCESS_TOKEN_KEY.len()..].into());
+                    return Some(stripped.into());
                 }
             }
         };
@@ -410,10 +410,12 @@ pub fn start_notification_server() -> WebSocketUsers {
 
     if CONFIG.websocket_enabled() {
         thread::spawn(move || {
-            let mut settings = ws::Settings::default();
+            let settings = ws::Settings {
-            settings.max_connections = 500;
+                max_connections: 500,
-            settings.queue_size = 2;
+                queue_size: 2,
-            settings.panic_on_internal = false;
+                panic_on_internal: false,
+                ..Default::default()
+            };
 
             ws::Builder::new()
                 .with_settings(settings)