Re: RFR 8048840: File.createTempFile has uninformative failure message

2014-07-02 Thread Alan Bateman
On 02/07/2014 01:02, Jeremy Manson wrote: Here you go (far too lazy to generate a webrev for this. :) ) 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 @@ -1997,8 +1997,10

Re: RFR 8048840: File.createTempFile has uninformative failure message

2014-07-01 Thread Stuart Marks
On 7/1/14 1:34 AM, Alan Bateman wrote: On 01/07/2014 02:21, Jeremy Manson wrote: 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 IOE

Re: RFR 8048840: File.createTempFile has uninformative failure message

2014-07-01 Thread Alan Bateman
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 IOExc

Re: RFR 8048840: File.createTempFile has uninformative failure message

2014-06-30 Thread Bernd
Hello, Do you think this fixes the complaints? I can imagine that "" or "~" is used, and including this in the exception does not really help. "Prefix string too short, must be 3 characters" would be my choice, but I wonder if the restriction is very usefull anyway? Bernd Am 01.07.2014 03:17 sch

Re: RFR 8048840: File.createTempFile has uninformative failure message

2014-06-30 Thread Jeremy Manson
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() <