From 519a960b55b2b752c9484912a02b02955c1e8e63 Mon Sep 17 00:00:00 2001
From: David Hewitt <mail@davidhewitt.dev>
Date: Fri, 10 Oct 2025 11:20:39 +0100
Subject: [PATCH] fix `clippy::manual_let_else` lint

---
 Cargo.toml                                | 1 -
 src/lookup_key.rs                         | 5 ++---
 src/serializers/fields.rs                 | 8 ++------
 src/serializers/type_serializers/tuple.rs | 5 ++---
 src/validators/datetime.rs                | 5 ++---
 src/validators/union.rs                   | 5 ++---
 6 files changed, 10 insertions(+), 19 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index df9003113..a8f1bb057 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -90,7 +90,6 @@ doc_markdown = "allow"
 float_cmp = "allow"
 fn_params_excessive_bools = "allow"
 if_not_else = "allow"
-manual_let_else = "allow"
 match_bool = "allow"
 match_same_arms = "allow"
 missing_errors_doc = "allow"
diff --git a/src/lookup_key.rs b/src/lookup_key.rs
index db8a4c869..7fe5d61e9 100644
--- a/src/lookup_key.rs
+++ b/src/lookup_key.rs
@@ -59,9 +59,8 @@ impl LookupKey {
             }
         } else {
             let list = value.downcast::<PyList>()?;
-            let first = match list.get_item(0) {
-                Ok(v) => v,
-                Err(_) => return py_schema_err!("Lookup paths should have at least one element"),
+            let Ok(first) = list.get_item(0) else {
+                return py_schema_err!("Lookup paths should have at least one element");
             };
             let mut locs: Vec<LookupPath> = if first.downcast::<PyString>().is_ok() {
                 // list of strings rather than list of lists
diff --git a/src/serializers/fields.rs b/src/serializers/fields.rs
index cf38eb40d..6065ad536 100644
--- a/src/serializers/fields.rs
+++ b/src/serializers/fields.rs
@@ -366,9 +366,7 @@ impl TypeSerializer for GeneralFieldsSerializer {
         // If there is no model, we (a TypedDict) are the model
         let model = extra.model.map_or_else(|| Some(value), Some);
 
-        let (main_dict, extra_dict) = if let Some(main_extra_dict) = self.extract_dicts(value) {
-            main_extra_dict
-        } else {
+        let Some((main_dict, extra_dict)) = self.extract_dicts(value) else {
             extra.warnings.on_fallback_py(self.get_name(), value, extra)?;
             return infer_to_python(value, include, exclude, extra);
         };
@@ -412,9 +410,7 @@ impl TypeSerializer for GeneralFieldsSerializer {
         exclude: Option<&Bound<'_, PyAny>>,
         extra: &Extra,
     ) -> Result<S::Ok, S::Error> {
-        let (main_dict, extra_dict) = if let Some(main_extra_dict) = self.extract_dicts(value) {
-            main_extra_dict
-        } else {
+        let Some((main_dict, extra_dict)) = self.extract_dicts(value) else {
             extra.warnings.on_fallback_ser::<S>(self.get_name(), value, extra)?;
             return infer_serialize(value, serializer, include, exclude, extra);
         };
diff --git a/src/serializers/type_serializers/tuple.rs b/src/serializers/type_serializers/tuple.rs
index 835d32544..20315e8a4 100644
--- a/src/serializers/type_serializers/tuple.rs
+++ b/src/serializers/type_serializers/tuple.rs
@@ -187,9 +187,8 @@ impl TupleSerializer {
         macro_rules! use_serializers {
             ($serializers_iter:expr) => {
                 for (index, serializer) in $serializers_iter.enumerate() {
-                    let element = match py_tuple_iter.next() {
-                        Some(value) => value,
-                        None => break,
+                    let Some(element) = py_tuple_iter.next() else {
+                        break;
                     };
                     let op_next = self.filter.index_filter(index, include, exclude, Some(n_items))?;
                     if let Some((next_include, next_exclude)) = op_next {
diff --git a/src/validators/datetime.rs b/src/validators/datetime.rs
index 23fdfe6a4..4c6810d4c 100644
--- a/src/validators/datetime.rs
+++ b/src/validators/datetime.rs
@@ -302,9 +302,8 @@ impl TZConstraint {
 
     pub(super) fn from_py(schema: &Bound<'_, PyDict>) -> PyResult<Option<Self>> {
         let py = schema.py();
-        let tz_constraint = match schema.get_item(intern!(py, "tz_constraint"))? {
-            Some(c) => c,
-            None => return Ok(None),
+        let Some(tz_constraint) = schema.get_item(intern!(py, "tz_constraint"))? else {
+            return Ok(None);
         };
         if let Ok(s) = tz_constraint.downcast::<PyString>() {
             let s = s.to_str()?;
diff --git a/src/validators/union.rs b/src/validators/union.rs
index 200e2b339..3b88bbc1d 100644
--- a/src/validators/union.rs
+++ b/src/validators/union.rs
@@ -357,9 +357,8 @@ impl Validator for TaggedUnionValidator {
                 let dict = input.validate_model_fields(state.strict_or(false), from_attributes)?;
                 // note this methods returns PyResult<Option<(data, data)>>, the outer Err is just for
                 // errors when getting attributes which should be "raised"
-                let tag = match dict.get_item(lookup_key)? {
-                    Some((_, value)) => value,
-                    None => return Err(self.tag_not_found(input)),
+                let Some((_, tag)) = dict.get_item(lookup_key)? else {
+                    return Err(self.tag_not_found(input));
                 };
                 self.find_call_validator(py, &tag.borrow_input().to_object(py)?, input, state)
             }