Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Brian" == Brian Jones <[EMAIL PROTECTED]> writes:
>
> >> this(dirname == null?name:dirname + separator + name);
>
> Brian> I implemented it this way instead of the way you later
> Brian> submitted. My reasoning here is that if name is null, the vm
> Brian> will naturally throw a NullPointerException at this point
> Brian> without us having to explicitly check and throw it ourselves.
>
> `+' is well-defined if either argument is null. JLS 2nd ed., section
> 15.18.1.
>
> You have to explicitly check at some point.
Thanks, this was in 15.17.1 in the version of the JLS I'm looking
at... probably an old version. Anyway you're right and the String
conversion leads to "null". I'll make a small change to correct the
previous commit.
Brian
--
Brian Jones <[EMAIL PROTECTED]>
_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath
