On 01/07/2014 02:21, Jeremy Manson wrote:
Oops - forgot to run jtreg. Make that:

diff --git a/src/share/classes/java/io/File.java
b/src/share/classes/java/io/File.java
--- a/src/share/classes/java/io/File.java
+++ b/src/share/classes/java/io/File.java
@@ -1998,7 +1998,8 @@
          throws IOException
      {
          if (prefix.length() < 3)
-            throw new IllegalArgumentException("Prefix string too short");
+            throw new IllegalArgumentException("Prefix string too short: "
+
+                                               prefix);
I assume you meant to "+ prefix" here, in which case the change seems okay to me. The update to the NulFile test seems okay too.

As a side point then I think the need for a prefix is somewhat legacy now and this API could be re-visited some day.

-Alan.

Reply via email to