Re: [cp-patches] FYI: StringTokenizer fixlet

[Tom Tromey]

>>>>> "Roman" == Roman Kennke <address@hidden> writes:

Roman> In the java.util.StringTokenizer we trigger an NPE if delim is null. The
Roman> spec explicitly says, that we shouldn't do so, and Sun's impl
Roman> doesn't. This patch fixes this.

I didn't look, but a mauve test for this would be great.

Roman> 2006-01-10  Roman Kennke  <address@hidden>
Roman>         * java/util/StringTokenizer.java
Roman>         (StringTokenizer(String, String, boolean)):
Roman>         Don't trigger NPE here for conformance with the spec.

I think the javadoc needs to be updated as well.  In my copy a couple
of the constructors mention this NPE:

   * @throws NullPointerException if either argument is null
   * @throws NullPointerException if str or delim is null

Tom