replace "".to_string() with String::new()

Author: Kn4ughty

src/apps/desktop_entry.rs

@@ -90,14 +90,14 @@ impl std::default::Default for DesktopEntry {
         DesktopEntry {
             entry_type: EntryType::Application,
             version: None,
-            name: "".to_string(),
+            name: String::new(),
             generic_name: None,
             comment: None,
             icon: None,
             only_show_in: Vec::new(),
             not_show_in: Vec::new(),
             try_exec: None,
-            exec: "".to_string(),
+            exec: String::new(),
             working_dir: None,
             terminal: false,
             action_list: Vec::new(),
@@ -322,14 +322,14 @@ impl From<DesktopEntry> for App {
 
                 log::trace!("arg is: {:#?}", arg);
 
-                if arg == vec!["".to_string()] {
+                if arg == vec![String::new()] {
                     log::trace!("ARGS LEN 0");
                     arg.clear();
                 }
 
                 (cmd.to_string(), arg)
             }
-            None => (desktop_entry.exec, vec!["".to_string()]),
+            None => (desktop_entry.exec, vec![String::new()]),
         };
 
         let working_dir = desktop_entry.working_dir;
@@ -405,7 +405,7 @@ fn can_parse_string_list() {
 
 fn parse_exec_key(input: &str, icon: Option<&str>, name: Option<&str>) -> String {
     // https://specifications.freedesktop.org/desktop-entry-spec/latest/exec-variables.html
-    let mut escaped_result = "".to_string();
+    let mut escaped_result = String::new();
     let mut chars = input.chars().peekable();
 
     // Lots of nesting here..

src/calculator/mod.rs

@@ -166,7 +166,7 @@ impl Calc {
         let mut out = Vec::new();
         let chars = source.chars();
 
-        let mut number_buf: String = "".to_string();
+        let mut number_buf: String = String::new();
 
         let mut iter = chars.enumerate().peekable();
         while let Some((idx, c)) = iter.next() {

src/main.rs

@@ -46,11 +46,11 @@ impl State {
 
         modules.insert("!".to_string(), LazyCell::new(|| Box::new(Web::new())));
 
-        modules.insert("".to_string(), LazyCell::new(|| Box::new(AppModule::new())));
+        modules.insert(String::new(), LazyCell::new(|| Box::new(AppModule::new())));
 
         log::info!("Time to initialise modules: {:#?}", start.elapsed());
         State {
-            text_value: "".to_string(),
+            text_value: String::new(),
             text_id: widget::Id::new("text_entry"),
             window_id: None,
             has_user_typed: false,
@@ -63,7 +63,7 @@ impl State {
         let mut modules: HashMap<String, LazyCell<Box<dyn Module>>> = HashMap::new();
 
         modules.insert(
-            "".to_string(),
+            String::new(),
             LazyCell::new(|| {
                 let stdin = std::io::stdin();
                 let mut lines = Vec::new();
@@ -80,7 +80,7 @@ impl State {
         start.finish();
 
         State {
-            text_value: "".to_string(),
+            text_value: String::new(),
             text_id: widget::Id::new("text_entry"),
             window_id: None,
             has_user_typed: false,