Skip to content

fix(style): nightly clippy#99

Merged
KillingSpark merged 2 commits intoKillingSpark:masterfrom
zleyyij:clippy-unwrap-is_some
Jan 8, 2026
Merged

fix(style): nightly clippy#99
KillingSpark merged 2 commits intoKillingSpark:masterfrom
zleyyij:clippy-unwrap-is_some

Conversation

@zleyyij
Copy link
Copy Markdown
Contributor

@zleyyij zleyyij commented Jan 3, 2026

This change makes nightly clippy happy by making the below style change where necessary:

-        let ll_code = if scratch.ll_rle.is_some() {
-            scratch.ll_rle.unwrap()
+        let ll_code = if let Some(ll_rle) = scratch.ll_rle {
+            ll_rle
         } else {
             ll_dec.decode_symbol()
         };

It also removes unix specific dependencies for the CLI so that it works on Windows.

@KillingSpark
Copy link
Copy Markdown
Owner

Oh nice, thanks!

@KillingSpark KillingSpark merged commit a39b341 into KillingSpark:master Jan 8, 2026
2 checks passed
zleyyij added a commit to zleyyij/zstd-rs that referenced this pull request Feb 20, 2026
* style(sequence decoder): migrate .is_some() -> .unwrap to if let to make clippy happy

* feat(cli): replace unix specific code with os generic variant
KillingSpark pushed a commit that referenced this pull request Feb 27, 2026
* meta: specify rust-version and add cargo-msrv to CI

* fix(style): nightly clippy (#99)

* style(sequence decoder): migrate .is_some() -> .unwrap to if let to make clippy happy

* feat(cli): replace unix specific code with os generic variant

* style: nightly clippy

---------

Co-authored-by: arc <zleyyij@users.noreply.github.com>
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