Skip to content

Commit 0ef2b32

Browse files
fix: Address clippy warnings
- Use if-let instead of match for single pattern - Replace useless format! with to_string()
1 parent f212565 commit 0ef2b32

2 files changed

Lines changed: 12 additions & 15 deletions

File tree

src/register/register_screen.rs

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -575,21 +575,18 @@ impl MatchEvent for RegisterScreen {
575575
_ => {}
576576
}
577577

578-
match action.downcast_ref::<RegisterAction>() {
579-
Some(RegisterAction::RegistrationSuccess | RegisterAction::RegistrationFailure(_)) => {
580-
// Always hide modal regardless of result
581-
self.view.modal(id!(status_modal)).close(cx);
582-
583-
// Re-enable register button for failure case (success will hide the screen)
584-
if matches!(action.downcast_ref::<RegisterAction>(), Some(RegisterAction::RegistrationFailure(_))) {
585-
let register_button = self.view.button(id!(register_button));
586-
register_button.set_enabled(cx, true);
587-
register_button.reset_hover(cx);
588-
}
589-
590-
self.redraw(cx);
578+
if let Some(RegisterAction::RegistrationSuccess | RegisterAction::RegistrationFailure(_)) = action.downcast_ref::<RegisterAction>() {
579+
// Always hide modal regardless of result
580+
self.view.modal(id!(status_modal)).close(cx);
581+
582+
// Re-enable register button for failure case (success will hide the screen)
583+
if matches!(action.downcast_ref::<RegisterAction>(), Some(RegisterAction::RegistrationFailure(_))) {
584+
let register_button = self.view.button(id!(register_button));
585+
register_button.set_enabled(cx, true);
586+
register_button.reset_hover(cx);
591587
}
592-
_ => {}
588+
589+
self.redraw(cx);
593590
}
594591
}
595592
}

src/sliding_sync.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -336,7 +336,7 @@ async fn register_user(register_request: RegisterRequest) -> Result<(Client, Cli
336336
// This avoids duplicate session persistence
337337
Ok((client, client_session))
338338
} else {
339-
let err_msg = format!("Registration completed but user is not logged in");
339+
let err_msg = "Registration completed but user is not logged in".to_string();
340340
enqueue_popup_notification(PopupItem {
341341
message: err_msg.clone(),
342342
kind: PopupKind::Error,

0 commit comments

Comments
 (0)