Bulding Warning: warning: function set_mobile_config_dir`is never used #183

clintre commented 2026年06月28日 16:53:41 +02:00

Nothing serious, just a warning. It has been there from the start and not overly concerned being a warning, but just wanted to let you know. When building for Linux...

warning: function `set_mobile_config_dir` is never used
 --> src/commands.rs:2427:8
 |
2427 | pub fn set_mobile_config_dir(path: std::path::PathBuf) {
 | ^^^^^^^^^^^^^^^^^^^^^
 |
 = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
 Building [=======================> ] 811/813: helixnotes 

Nothing serious, just a warning. It has been there from the start and not overly concerned being a warning, but just wanted to let you know. When building for Linux... ``` warning: function `set_mobile_config_dir` is never used --> src/commands.rs:2427:8 | 2427 | pub fn set_mobile_config_dir(path: std::path::PathBuf) { | ^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default Building [=======================> ] 811/813: helixnotes ```

ArkHost commented 2026年06月28日 17:01:10 +02:00

Thanks for the heads-up. It's harmless: set_mobile_config_dir is an Android-only entry point, so on desktop builds it's just dead code and Rust flags it. Nothing broken, the build and runtime are fine.

I'll gate it behind the #[cfg(target_os = "android")] in a coming release so the warning clears. No rush since it's cosmetic. Thanks for keeping HelixNotes in Solus!

Thanks for the heads-up. It's harmless: set_mobile_config_dir is an Android-only entry point, so on desktop builds it's just dead code and Rust flags it. Nothing broken, the build and runtime are fine. I'll gate it behind the #[cfg(target_os = "android")] in a coming release so the warning clears. No rush since it's cosmetic. Thanks for keeping HelixNotes in Solus!

clintre commented 2026年06月28日 17:02:53 +02:00

I figured it was not more of a heads-up than anything. It has no effect on me.

I figured it was not more of a heads-up than anything. It has no effect on me.