refactor: Implement Default for SegmentConfig (#495)

Implements the Default trait for SegmentConfig to clean up construction of empty segments.

Also adds a segment::new() function to ease construction of simple segments.

src/config.rs

@@ -219,6 +219,10 @@ impl<'a> ModuleConfig<'a> for SegmentConfig<'a> {
 }
 
 impl<'a> SegmentConfig<'a> {
+    pub fn new(value: &'static str) -> Self {
+        Self { value, style: None }
+    }
+
     /// Mutably set value
     pub fn set_value(&mut self, value: &'a str) {
         self.value = value;
@@ -246,6 +250,12 @@ impl<'a> SegmentConfig<'a> {
     }
 }
 
+impl Default for SegmentConfig<'static> {
+    fn default() -> Self {
+        Self::new("")
+    }
+}
+
 /** Parse a style string which represents an ansi style. Valid tokens in the style
  string include the following:
  - 'fg:<color>'    (specifies that the color read should be a foreground color)

src/configs/aws.rs

@@ -14,14 +14,8 @@ pub struct AwsConfig<'a> {
 impl<'a> RootModuleConfig<'a> for AwsConfig<'a> {
     fn new() -> Self {
         AwsConfig {
-            symbol: SegmentConfig {
+            symbol: SegmentConfig::new("☁️ "),
-                value: "☁️ ",
+            profile: SegmentConfig::default(),
-                style: None,
-            },
-            profile: SegmentConfig {
-                value: "",
-                style: None,
-            },
             style: Color::Yellow.bold(),
             disabled: false,
         }

src/configs/battery.rs

@@ -18,18 +18,9 @@ pub struct BatteryConfig<'a> {
 impl<'a> RootModuleConfig<'a> for BatteryConfig<'a> {
     fn new() -> Self {
         BatteryConfig {
-            full_symbol: SegmentConfig {
+            full_symbol: SegmentConfig::new("•"),
-                value: "•",
+            charging_symbol: SegmentConfig::new("↑"),
-                style: None,
+            discharging_symbol: SegmentConfig::new("↓"),
-            },
-            charging_symbol: SegmentConfig {
-                value: "↑",
-                style: None,
-            },
-            discharging_symbol: SegmentConfig {
-                value: "↓",
-                style: None,
-            },
             unknown_symbol: None,
             empty_symbol: None,
             display: vec![BatteryDisplayConfig {
@@ -37,10 +28,7 @@ impl<'a> RootModuleConfig<'a> for BatteryConfig<'a> {
                 style: Color::Red.bold(),
             }],
             disabled: false,
-            percentage: SegmentConfig {
+            percentage: SegmentConfig::default(),
-                value: "",
-                style: None,
-            },
         }
     }
 }

src/configs/character.rs

@@ -17,18 +17,9 @@ pub struct CharacterConfig<'a> {
 impl<'a> RootModuleConfig<'a> for CharacterConfig<'a> {
     fn new() -> Self {
         CharacterConfig {
-            symbol: SegmentConfig {
+            symbol: SegmentConfig::new("❯"),
-                value: "❯",
+            error_symbol: SegmentConfig::new("✖"),
-                style: None,
+            vicmd_symbol: SegmentConfig::new("❮"),
-            },
-            error_symbol: SegmentConfig {
-                value: "✖",
-                style: None,
-            },
-            vicmd_symbol: SegmentConfig {
-                value: "❮",
-                style: None,
-            },
             use_symbol_for_status: false,
             style_success: Color::Green.bold(),
             style_failure: Color::Red.bold(),

src/configs/dotnet.rs

@@ -15,14 +15,8 @@ pub struct DotnetConfig<'a> {
 impl<'a> RootModuleConfig<'a> for DotnetConfig<'a> {
     fn new() -> Self {
         DotnetConfig {
-            symbol: SegmentConfig {
+            symbol: SegmentConfig::new("•NET "),
-                value: "•NET ",
+            version: SegmentConfig::default(),
-                style: None,
-            },
-            version: SegmentConfig {
-                value: "",
-                style: None,
-            },
             style: Color::Blue.bold(),
             heuristic: true,
             disabled: false,

src/configs/kubernetes.rs

@@ -15,18 +15,9 @@ pub struct KubernetesConfig<'a> {
 impl<'a> RootModuleConfig<'a> for KubernetesConfig<'a> {
     fn new() -> Self {
         KubernetesConfig {
-            symbol: SegmentConfig {
+            symbol: SegmentConfig::new("☸ "),
-                value: "☸ ",
+            context: SegmentConfig::default(),
-                style: None,
+            namespace: SegmentConfig::default(),
-            },
-            context: SegmentConfig {
-                value: "",
-                style: None,
-            },
-            namespace: SegmentConfig {
-                value: "",
-                style: None,
-            },
             style: Color::Cyan.bold(),
             disabled: true,
         }

src/configs/rust.rs

@@ -14,14 +14,8 @@ pub struct RustConfig<'a> {
 impl<'a> RootModuleConfig<'a> for RustConfig<'a> {
     fn new() -> Self {
         RustConfig {
-            symbol: SegmentConfig {
+            symbol: SegmentConfig::new("🦀 "),
-                value: "🦀 ",
+            version: SegmentConfig::default(),
-                style: None,
-            },
-            version: SegmentConfig {
-                value: "",
-                style: None,
-            },
             style: Color::Red.bold(),
             disabled: false,
         }