minor error reporting patch

Mon Jun 25 20:26:00 GMT 2001

>>>>> "David" == David Brownell <david-b@pacbell.net> writes:

David> Running that API test suite, I noticed that file open errors
David> gave a different exception string. That shouldn't be a
David> big deal, except that since Java doesn't really provide good
David> ways to distinguish many errors except by such strings,
David> programs often end up needing to compare them.
This is not really something I like to do, since it means we're
indirectly supporting bad programming habits. However this particular
patch is pretty innocuous, and it is hard to argue against it, so I am
going to check it in.
There are certain kinds of bug compatibility changes we probably ought
to reject. We can do that on a case-by-case basis though.
David> --- natFdOrig.cc Mon Jun 25 11:14:44 2001
David> +++ natFileDescriptorPosix.cc Mon Jun 25 11:15:17 2001
David> @@ -111,7 +111,7 @@
David> if (fd == -1)
David> {
David> char msg[MAXPATHLEN + 200];
David> - sprintf (msg, "%s: %s", buf, strerror (errno));
David> + sprintf (msg, "%s (%s)", buf, strerror (errno));
David> JvThrow (new FileNotFoundException (JvNewStringLatin1 (msg)));
David> }
David> return fd;
Thanks,
Tom