Hello Pekka,

Mon, 6 Dec 2010 11:36:06 +0200 Pekka Enberg <penb...@kernel.org>:

> Hi all,
> 
> I added the following PrintStream constructor API tests to Malva:
> 
> https://github.com/penberg/malva/commit/b5020c862656c491fc81b2f4f3cee39a6444709a
> 
> Some of the tests fail with GNU Classpath but Ivan's patch from June
> cures most of the issues:
> 
> http://developer.classpath.org/pipermail/classpath-patches/2010-June/006410.html
> 
> [ There's still constructor throwing IllegalArgumentException where
> NPE happens with Oracle JDK. ]
> 
> The patch looks good to me apart from the "line_separator" static
> modifier removal so FWIW,

line_sepator issue (as well as the constructor NPE on null out) has already 
been discussed 
(http://article.gmane.org/gmane.comp.java.classpath.patches/12976/)

> 
> Reviewed-by: Pekka Enberg <penb...@kernel.org>
> 
> Pekka

Regards.

Reply via email to