Merge pull request #20 from xadrianzetx/fix-delete-types

Pass `name` as string slice to `delete_game_by_name`

Author: xadrianzetx

src/commands/delete.rs

@@ -5,7 +5,7 @@ use std::path::Path;
 
 use clap::ArgMatches;
 
-fn delete_game_by_name(path: &Path, name: String) -> Result<()> {
+fn delete_game_by_name(path: &Path, name: &str) -> Result<()> {
     let ulpath = path.join(Path::new("ul.cfg"));
     let mut ulcfg = Ulcfg::load(&ulpath)?;
     ulcfg.delete_game_by_name(name)?;
@@ -34,8 +34,7 @@ pub fn delete(args: &ArgMatches) {
     }
 
     if let Some(name) = args.value_of("name") {
-        let namestr = String::from(name);
-        match delete_game_by_name(path, namestr) {
+        match delete_game_by_name(path, name) {
             Ok(()) => (),
             Err(_) => println!("Could not delete game by name"),
         }

src/ul/mod.rs

@@ -128,9 +128,9 @@ impl Ulcfg {
         Ok(())
     }
 
-    pub fn delete_game_by_name(&mut self, name: String) -> Result<()> {
+    pub fn delete_game_by_name(&mut self, name: &str) -> Result<()> {
         for (index, game) in self.games.iter().enumerate() {
-            if game.opl_name == name.as_str() {
+            if game.opl_name == name {
                 self.delete_game(index)?;
                 return Ok(());
             }