refactoring: use String instead of &str in interpret

src/execute.rs

@@ -5,11 +5,11 @@ use crate::builtins::{execute_builtin, is_builtin};
 use crate::error::ShellError;
 use crate::parse::parse_line;
 
-pub fn interpret(line: &str) -> Result<(), ShellError> {
+pub fn interpret(line: String) -> Result<(), ShellError> {
     if line.is_empty() {
         return Err(ShellError::EmptyLine);
     }
-    let (keyword, args) = parse_line(line)?;
+    let (keyword, args) = parse_line(&line)?;
 
     if is_builtin(keyword) {
         execute_builtin(keyword, args)?;

src/main.rs

@@ -20,7 +20,7 @@ fn main() -> Result<()> {
         let readline = rl.readline(&prompt.get_prompt());
 
         match readline {
-            Ok(line) => match interpret(&line) {
+            Ok(line) => match interpret(line) {
                 Ok(_) => {}
                 Err(ShellError::EmptyLine) => {
                     continue;