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