On Mon, May 21, 2018 at 03:01:57PM +0200, Cyril Ferlicot D. wrote:
> Le 21/05/2018 ?? 14:58, Alistair Grant a ??crit??:
> > Hi Everyone,
> >
> > Does anyone know the history behind FileDoesNotExist and
> > FileDoesNotExistException?
> >
> > Both classes exist in 6.1, and as far as I can tell, both are intended
> > to do the same thing, i.e. the class comments are:
> >
> > FileDoesNotExist;
> >
> > I am raised when an operation is attempted on a file that does not
> > exist. This includes cases where a file operation is attempted on a
> > directory.
> >
> >
> >
> > FileDoesNotExistException:
> >
> > Notify when fie does not exist
> >
> >
> >
> > This means that programs have to check for both in exceptions, or be
> > very sure of whether any object they are using ultimately refers to a
> > File or a FileReference.
> >
> > If there isn't a good reason for having both, I think we should remove
> > one.�� Since FileDoesNotExistException is loaded first in the bootstrap
> > process (since it is part of Files, not FileSystem), keeping it probably
> > makes more sense.
> >
> > Similar duplication exists for FileAlreadyExistsException and
> > FileExists.
> >
> > Thoughts?
> >
>
> Hi Alistar,
>
> See the conversation here:
>
> https://pharo.fogbugz.com/f/cases/19026/Merge-FileDoesNotExist-and-FileDoesNotExistException

Thanks, Cyril!



On Mon, May 21, 2018 at 03:36:05PM +0200, Sven Van Caekenberghe wrote:
> I vote for the simple names without Exception in them.

On Mon, May 21, 2018 at 05:59:32PM +0300, Denis Kudriashov wrote:
> +1

Me too... :-)


I'll work on this next.

Cheers,
Alistair