If I'm recalling well (which I might not :)) there is one thing (that I don't quite like) that prevents these two classes to be easily merged into one: their #defaultAction.
On Mon, May 21, 2018 at 7:07 PM, Alistair Grant <[email protected]> wrote: > 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 > > > -- Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - *http://www.cnrs.fr <http://www.cnrs.fr>* *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13
