Most of this is just allowing "implicit" conversions between
Path/PathBuf and ErrorKind, enabling `.chain_err(|| path )` syntax as
opposed to `.chain_err(|| ErrorKind::Path(path.to_path_buf()) )`.
Reduced verbosity should help with readability, since that pattern is
used throughout the codebase.

Also took care of a couple TODOs