[m-rev.] for review: Improve io.file.m for Java and C#.
Peter Wang
novalazy at gmail.com
Thu Aug 25 11:38:52 AEST 2022
On Wed, 24 Aug 2022 19:16:28 +1000 Julien Fischer <jfischer at opturion.com> wrote:
>
> Hi Peter,
>
> On Wed, 24 Aug 2022, Peter Wang wrote:
>
> Start the log message with:
>
> Improve io.file.m for Java and C#.
>
> otherwise things will look odd in git log.
>
The summary line of the git commit message is contained in the subject
of the email message (created by git send-email). Would the git log
still look odd then?
I've added an introductory paragraph anyway.
> > diff --git a/library/io.file.m b/library/io.file.m
> > index 22a01a22f..51427bcd9 100644
> > --- a/library/io.file.m
> > +++ b/library/io.file.m
> > + // This first test just improves the error message in a common case.
> > + if (!old_file.exists()) {
> > + // java.io.FileNotFoundException is documented as being thrown when
> > + // failing to open a file but I don't see any reason we cannot use
> > + // it in this case. (nio also defines a NoSuchFileException class.)
>
> In principle, there's no reason to avoid stuff from the java.nio
> package. (Of course, since we currently don't really use it in the io module,
> grabbing the exceptions from there might look a bit odd.)
README.Java says we require Java SE 8. Does that mean we can start using nio?
Peter
More information about the reviews
mailing list