Xueming Shen wrote:
Alan,

Can you help review?

(1)TestUTF8
This one is no longer reproducible, it passes on all platforms during the last several jprt runs.

(2)FindDecoderBugs
The only possible cause of its failure on a particular platform is timeout, this test takes long cpu time on slow machine. I tweak it a little, it runs about 30% faster now. It passes the last several jprt runs
   on all platforms.

(3) TestX11CNS
This test is not really necessary (it is useful when migration to new implementation) but I modified
     it to run and pass anyway.

http://cr.openjdk.java.net/~sherman/test_cs/webrev/
Speeding up FindDecoderBugs is good. Is the static equals method that you've added to the "Infrastructure" section used?

TestX11CNS.getCharsets might consume an exception that we might want to see. Would it be better if it only returned null if UnsupportedCharsetException is thrown, and maybe have the test fail if there is another problem.

Do you have cycles to take a pass over the sun/nio/cs tests to see if they could run in samevm mode? I think they are in batch jdk_nio3 at the moment and that test target currently runs in othervm mode.

-Alan.

Reply via email to