other: clean up some cfg usage (#1205)

* other: convert some cfg if

* fix

src/app/layout_manager.rs

@@ -1012,9 +1012,11 @@ impl std::str::FromStr for BottomWidgetType {
             "temp" | "temperature" => Ok(BottomWidgetType::Temp),
             "disk" => Ok(BottomWidgetType::Disk),
             "empty" => Ok(BottomWidgetType::Empty),
-            "battery" | "batt" if cfg!(feature = "battery") => Ok(BottomWidgetType::Battery),
+            #[cfg(feature = "battery")]
+            "battery" | "batt" => Ok(BottomWidgetType::Battery),
             _ => {
-                if cfg!(feature = "battery") {
+                #[cfg(feature = "battery")]
+                {
                     Err(BottomError::ConfigError(format!(
                         "\"{}\" is an invalid widget name.
         
@@ -1037,7 +1039,9 @@ Supported widget names:
                 ",
                         s
                     )))
-                } else {
+                }
+                #[cfg(not(feature = "battery"))]
+                {
                     Err(BottomError::ConfigError(format!(
                         "\"{}\" is an invalid widget name.
 

src/canvas/dialogs/dd_dialog.rs

@@ -247,6 +247,8 @@ impl Painter {
             );
 
             if app_state.should_get_widget_bounds() {
+                const SIGNAL: usize = if cfg!(target_os = "windows") { 1 } else { 15 };
+
                 // This is kinda weird, but the gist is:
                 // - We have three sections; we put our mouse bounding box for the "yes" button at the very right edge
                 //   of the left section and 3 characters back.  We then give it a buffer size of 1 on the x-coordinate.
@@ -263,7 +265,7 @@ impl Painter {
                         button_layout[0].y,
                         button_layout[0].x + button_layout[0].width,
                         button_layout[0].y,
-                        if cfg!(target_os = "windows") { 1 } else { 15 },
+                        SIGNAL,
                     ),
                     // No
                     (

src/canvas/widgets/cpu_graph.rs

@@ -1,6 +1,5 @@
 use std::borrow::Cow;
 
-use concat_string::concat_string;
 use tui::{
     backend::Backend,
     layout::{Constraint, Direction, Layout, Rect},
@@ -201,16 +200,21 @@ impl Painter {
             );
 
             // TODO: Maybe hide load avg if too long? Or maybe the CPU part.
-            let title = if cfg!(target_family = "unix") {
+            let title = {
+                #[cfg(target_family = "unix")]
+                {
                     let load_avg = app_state.converted_data.load_avg_data;
                     let load_avg_str = format!(
                         "─ {:.2} {:.2} {:.2} ",
                         load_avg[0], load_avg[1], load_avg[2]
                     );
 
-                concat_string!(" CPU ", load_avg_str).into()
+                    concat_string::concat_string!(" CPU ", load_avg_str).into()
-            } else {
+                }
+                #[cfg(not(target_family = "unix"))]
+                {
                     " CPU ".into()
+                }
             };
 
             let marker = if app_state.app_config_fields.use_dot {

tests/arg_tests.rs

@@ -165,7 +165,6 @@ fn test_missing_default_widget_type() {
 #[test]
 #[cfg_attr(feature = "battery", ignore)]
 fn test_battery_flag() {
-    if !cfg!(feature = "battery") {
     btm_command()
         .arg("--battery")
         .assert()
@@ -173,13 +172,11 @@ fn test_battery_flag() {
         .stderr(predicate::str::contains(
             "unexpected argument '--battery' found",
         ));
-    }
 }
 
 #[test]
 #[cfg_attr(feature = "gpu", ignore)]
 fn test_gpu_flag() {
-    if !cfg!(feature = "gpu") {
     btm_command()
         .arg("--enable_gpu_memory")
         .assert()
@@ -187,5 +184,4 @@ fn test_gpu_flag() {
         .stderr(predicate::str::contains(
             "unexpected argument '--enable_gpu_memory' found",
         ));
-    }
 }