From d30313ac4df8949ab2686edb84caf21105720554 Mon Sep 17 00:00:00 2001 From: Daniel McCarney Date: Thu, 9 Jul 2026 14:41:34 -0400 Subject: [PATCH] crl: fix clippy::question_mark finding Of the form: ``` error: this `match` expression can be replaced with `?` --> src/crl/types.rs:357:13 | 357 | / match revoked_cert_result { 358 | | Err(e) => return Err(e), 359 | | Ok(revoked_cert) => { 360 | | if revoked_cert.serial_number.eq(serial) { ... | 364 | | } | |_____________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.97.0/index.html#question_mark = note: `-D clippy::question-mark` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::question_mark)]` ``` --- src/crl/types.rs | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/crl/types.rs b/src/crl/types.rs index 8e8d1c59..3c311ddc 100644 --- a/src/crl/types.rs +++ b/src/crl/types.rs @@ -354,13 +354,9 @@ impl<'a> BorrowedCertRevocationList<'a> { fn find_serial(&self, serial: &[u8]) -> Result>, Error> { for revoked_cert_result in self { - match revoked_cert_result { - Err(e) => return Err(e), - Ok(revoked_cert) => { - if revoked_cert.serial_number.eq(serial) { - return Ok(Some(revoked_cert)); - } - } + let revoked_cert = revoked_cert_result?; + if revoked_cert.serial_number.eq(serial) { + return Ok(Some(revoked_cert)); } }