From ab803c07af86bac71667b7fe5098e7c8b9700d0c Mon Sep 17 00:00:00 2001 From: funman300 Date: Fri, 8 May 2026 21:53:36 -0700 Subject: [PATCH] fix(android): remove unused JValue import and fix match arm types Two cfg(android) issues hidden from Linux CI: - android_clipboard.rs: JValue was imported but never used (JValueOwned covers all call sites). Removed to satisfy -D unused-imports. - stats_plugin.rs: both arms of the clipboard match now return () via explicit block+semicolon, resolving the type mismatch that pinged-pong between runs due to bidirectional match-arm type inference. Co-Authored-By: Claude Sonnet 4.6 --- solitaire_engine/src/android_clipboard.rs | 2 +- solitaire_engine/src/stats_plugin.rs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/solitaire_engine/src/android_clipboard.rs b/solitaire_engine/src/android_clipboard.rs index 99cbcd5..f7c503d 100644 --- a/solitaire_engine/src/android_clipboard.rs +++ b/solitaire_engine/src/android_clipboard.rs @@ -6,7 +6,7 @@ pub fn set_text(text: &str) -> Result<(), String> { use bevy::android::ANDROID_APP; use jni::{ - objects::{JObject, JValue, JValueOwned}, + objects::{JObject, JValueOwned}, JavaVM, }; diff --git a/solitaire_engine/src/stats_plugin.rs b/solitaire_engine/src/stats_plugin.rs index 4a34267..28aa63c 100644 --- a/solitaire_engine/src/stats_plugin.rs +++ b/solitaire_engine/src/stats_plugin.rs @@ -362,10 +362,10 @@ fn handle_copy_share_link_button( #[cfg(target_os = "android")] { match crate::android_clipboard::set_text(&url) { - Ok(()) => toast.write(InfoToastEvent(format!("Copied: {url}"))), + Ok(()) => { toast.write(InfoToastEvent(format!("Copied: {url}"))); } Err(e) => { warn!("android clipboard failed: {e}"); - toast.write(InfoToastEvent(format!("Share link: {url}"))) + toast.write(InfoToastEvent(format!("Share link: {url}"))); } } }