Skip to content

fix: silence rdev compiler warnings#470

Open
mynameistito wants to merge 1 commit into
mulaRahul:mainfrom
mynameistito:fix/silence-rdev-compiler-warnings
Open

fix: silence rdev compiler warnings#470
mynameistito wants to merge 1 commit into
mulaRahul:mainfrom
mynameistito:fix/silence-rdev-compiler-warnings

Conversation

@mynameistito

@mynameistito mynameistito commented Apr 18, 2026

Copy link
Copy Markdown

Summary

Fixes: #468

  • android.rs: fix BackQuote keycode 75→68 (duplicate of Quote; correct is KEYCODE_GRAVE)
  • chrome.rs: add #[allow(unreachable_patterns)] — multiple keys legitimately map to "" in Chrome
  • windows/grab.rs, listen.rs: &mut GLOBAL_CALLBACK&mut *addr_of_mut!(GLOBAL_CALLBACK) to avoid static_mut_refs UB

Test plan

  • Build compiles without warnings on Windows, Android, Chrome targets (only tested win)

- android.rs: BackQuote keycode 75→68 (was duplicate of Quote/KEYCODE_APOSTROPHE; correct is KEYCODE_GRAVE)
- chrome.rs: add #[allow(unreachable_patterns)] to key_from_code — multiple keys legitimately map to "" in Chrome
- windows/grab.rs, listen.rs: &mut GLOBAL_CALLBACK → &mut *addr_of_mut!(GLOBAL_CALLBACK) to avoid static_mut_refs UB
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.

[Bug]: fix compiler warnings in the vendored rdev crate

1 participant