getPath opens the handle and throws any exception needed. So, it will
either return a good handle or throw the right exception, in which case it
is redundant to try/catch for it again.
 On Sep 14, 2013 8:19 AM, "Josh Albrecht" <notifications@github.com> wrote:

> In src/main/scala/com/codexica/s3crate/filetree/local/LinuxFileTree.scala:
>
> > @@ -19,7 +61,26 @@ class LinuxFileTree(baseFolder: File, ec: ExecutionContext) extends LocalFileTre
> >     * @throws InaccessibleDataError wrapping all other non fatal errors
> >     * @return Return the appropriate FilePathType (folder, file, or symlink)
> >     */
> > -  def getFileType(file: File): FilePathType = throw new NotImplementedError()
> > +  def getFileType(file: File): FilePathType = {
> > +    //get Path from File
> > +    val path = getPath(file)
>
> I'm curious why you left this outside of the try-except?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/joshalbrecht/s3crate/pull/1/files#r6364004>
> .
>