Skip to content

satisfy cargo clippy#4

Merged
eugenebokhan merged 1 commit into
masterfrom
fix-clippy-needless-late-init
Jul 6, 2026
Merged

satisfy cargo clippy#4
eugenebokhan merged 1 commit into
masterfrom
fix-clippy-needless-late-init

Conversation

@eugenebokhan

Copy link
Copy Markdown

Fixes the failing Clippy CI job (the only failing job on the scheduled master runs).

Nightly clippy's clippy::needless_late_init (part of clippy::all) now flags the late-initialized self_lt_token/self_gt_token bindings in macro/src/expand.rs, which aborts cxxbridge-macro and fails the job. Applied clippy's own suggestion: hoist the bindings into a tuple produced by the match. Behavior-preserving (the syn token fields are Copy).

Verified locally with the exact CI commands:

cargo +nightly clippy --workspace --tests --exclude demo -- -Dclippy::all -Dclippy::pedantic
cargo +nightly clippy --manifest-path demo/Cargo.toml -- -Dclippy::all

Both pass; with this lint fixed the whole workspace + tests are clippy-clean.

@eugenebokhan eugenebokhan merged commit b616fe2 into master Jul 6, 2026
51 checks passed
@eugenebokhan eugenebokhan deleted the fix-clippy-needless-late-init branch July 6, 2026 13:01
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.

1 participant