Fix formating and cleanup compiler warnings

src/builtin.rs

 use std::collections::HashMap;
-use std::fs::{self, File};
-use std::io::{stdout, Read, Write};
+use std::io::{stdout, Write};
 use std::env;
 use std::process;
-use std::thread;
 
-use super::to_num::ToNum;
 use super::set_var;
 use super::input_editor::readln;
 

src/directory_stack.rs

-use std::env::{set_current_dir,current_dir};
-use std::path::{PathBuf,Path};
-use std::ops::Deref;
+use std::env::{set_current_dir, current_dir};
+use std::path::PathBuf;
 
 pub struct DirectoryStack {
-    dirs: Vec<PathBuf>, // The top is 
+    dirs: Vec<PathBuf>, // The top is always the current directory
 }
 
-fn get_current_dir() -> PathBuf { // TODO STOP DOING THIS BAD STUFF
-    if let Ok(cur_dir) = current_dir() { 
-        cur_dir
-    } else {
-        PathBuf::new()
-    }
-}
-
-
 impl DirectoryStack {
-
     pub fn new() -> Result<DirectoryStack, &'static str> {
         if let Ok(curr_dir) = current_dir() {
-            Ok(DirectoryStack {
-                dirs: vec![curr_dir],
-            })
+            Ok(DirectoryStack { dirs: vec![curr_dir] })
         } else {
             Err("Failed to get current directory when building directory stack")
         }
     }
 
-    pub fn popd(&mut self, args: &[String]) {
-        if (self.dirs.len() < 2) {
+    pub fn popd(&mut self, _: &[String]) {
+        if self.dirs.len() < 2 {
             println!("Directory stack is empty");
             return;
         }
@@ -36,7 +23,7 @@ impl DirectoryStack {
             if let Err(err) = set_current_dir(dir) {
                 println!("{}: Failed to switch to directory {}", err, dir.display());
                 return;
-            } 
+            }
         }
         self.dirs.pop();
         self.print_dirs();
@@ -45,8 +32,13 @@ impl DirectoryStack {
     pub fn pushd(&mut self, args: &[String]) {
         if let Some(dir) = args.get(1) {
             match (set_current_dir(dir), current_dir()) {
-                (Ok(()), Ok(cur_dir)) => { self.dirs.push(cur_dir); },
-                (Err(err), _) => { println!("{}: {}", err, dir); return; },
+                (Ok(()), Ok(cur_dir)) => {
+                    self.dirs.push(cur_dir);
+                }
+                (Err(err), _) => {
+                    println!("{}: {}", err, dir);
+                    return;
+                }
                 (_, _) => (),
             }
         } else {
@@ -56,7 +48,7 @@ impl DirectoryStack {
         self.print_dirs();
     }
 
-    pub fn dirs(&self, args: &[String]) {
+    pub fn dirs(&self, _: &[String]) {
         self.print_dirs()
     }
 

src/main.rs

@@ -31,7 +31,6 @@ pub struct Shell {
 }
 
 impl Shell {
-
     /// Panics if DirectoryStack construction fails
     pub fn new() -> Shell {
         Shell {

src/tokenizer.rs

@@ -38,10 +38,7 @@ fn process_character_single_quoted(_: &mut Vec<Token>,
     }
 }
 
-fn process_character_comment(tokens: &mut Vec<Token>,
-                             current_token: &mut String,
-                             chr: char)
-                             -> TokenizerState {
+fn process_character_comment(tokens: &mut Vec<Token>, _: &mut String, chr: char) -> TokenizerState {
     match chr {
         '\n' | '\r' => {
             tokens.push(Token::End);