[cp-patches] Minor refactoring of PrintStream

Pekka Enberg penberg at kernel.org
Mon Dec 6 09:36:06 UTC 2010


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,

Reviewed-by: Pekka Enberg <penberg at kernel.org>

                        Pekka



More information about the Classpath-patches mailing list