"Wu, Gansha" <[EMAIL PROTECTED]> writes:
> We found a minor bug in java.io.File when we tried to run Volano benchmark on
>ORP with Classpath.
> Here's the senario:
> File file = new File(System.getProperty(propname), filename);
> While System.getProperty(propname) return null, then the constructor:
> File(String dirname, String name){
> this(dirname + separator + name);
> }
> assumes that a file with filename must exist under the directory named
>"null".
> An approperiate modification is to add a check like this:
> this(dirname == null?name:dirname + separator + name);
> The context compare result below is based on gnu classpath cvs snapshot of June
>22.Note that revision number is different from public CVS.
>
I implemented it this way instead of the way you later submitted. My
reasoning here is that if name is null, the vm will naturally throw a
NullPointerException at this point without us having to explicitly
check and throw it ourselves.
Brian
--
Brian Jones <[EMAIL PROTECTED]>
_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath
