commit f0906f4ddc1e1ae449981979e9fc9691189f6d66
parent 16bd4d80ff8d23542a752f948223113306aa6ee3
Author: William Casarin <jb55@jb55.com>
Date: Wed, 18 Feb 2026 12:40:10 -0800
fix clippy lints: too_many_arguments and question_mark
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Diffstat:
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/crates/notedeck_dave/src/session_events.rs b/crates/notedeck_dave/src/session_events.rs
@@ -517,6 +517,7 @@ fn build_single_event(
/// from role + content strings. No kind-1989 source-data events are created.
///
/// Calls `threading.record()` internally.
+#[allow(clippy::too_many_arguments)]
pub fn build_live_event(
content: &str,
role: &str,
diff --git a/crates/notedeck_dave/src/update.rs b/crates/notedeck_dave/src/update.rs
@@ -198,9 +198,7 @@ pub fn handle_permission_response(
request_id: uuid::Uuid,
response: PermissionResponse,
) -> Option<PermissionPublish> {
- let Some(session) = session_manager.get_active_mut() else {
- return None;
- };
+ let session = session_manager.get_active_mut()?;
let is_remote = session.is_remote();
@@ -253,9 +251,7 @@ pub fn handle_question_response(
request_id: uuid::Uuid,
answers: Vec<QuestionAnswer>,
) -> Option<PermissionPublish> {
- let Some(session) = session_manager.get_active_mut() else {
- return None;
- };
+ let session = session_manager.get_active_mut()?;
let is_remote = session.is_remote();