Skip to content

apply clippy fixes - #2417

Merged
baloo merged 2 commits into
RustCrypto:masterfrom
dishmaker:dishmaker/clippy_100_fixes
Aug 31, 2026
Merged

apply clippy fixes#2417
baloo merged 2 commits into
RustCrypto:masterfrom
dishmaker:dishmaker/clippy_100_fixes

Conversation

@dishmaker

Copy link
Copy Markdown
Contributor

pkcs12, x509-cert: clippy 0.1.100 (17fd5b8a37 2026-08-28)

warning: lint `clippy::from_iter_instead_of_collect` has been removed: lint has proved problematic
  |
  = note: requested on the command line with `-W clippy::from_iter_instead_of_collect`
  = note: `#[warn(renamed_and_removed_lints)]` on by default




warning: unneeded `return` statement with `?` operator
  --> x509-cert/src/ext/pkix/sct.rs:73:13
   |
73 |             return Err(tls_codec::Error::TrailingData)?;
   |             ^^^^^^^ help: remove it
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/main/index.html#needless_return_with_question_mark
   = note: `#[warn(clippy::needless_return_with_question_mark)]` on by default



warning: this `match` expression can be replaced with `?`
  --> pkcs12/src/safe_bag.rs:47:25
   |
47 |           let bag_value = match reader.tlv_bytes() {
   |  _________________________^
48 | |             Ok(v) => v.to_vec(),
49 | |             Err(e) => return Err(e),
50 | |         };
   | |_________^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/main/index.html#question_mark
   = note: `#[warn(clippy::question_mark)]` on by default
help: try instead
   |
47 ~         let bag_value = {
48 +             let v = reader.tlv_bytes()?;
49 +             v.to_vec()
50 ~         };
   |

pkcs12, x509-cert: clippy 0.1.100 (17fd5b8a37 2026-08-28)
@dishmaker
dishmaker marked this pull request as ready for review August 29, 2026 17:48
@baloo
baloo merged commit e90fd35 into RustCrypto:master Aug 31, 2026
189 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants