Hi Christoph, I updated webrev: http://cr.openjdk.java.net/~horii/8179527/webrev.02/
This one includes changes in tests. I removed all @run and @build directives in the tests because those after removing "@run main/othervm -Dsun.nio.cs.bugLevel=1.4 EmptyCharsetName" are the same as the default ones. I checked the modified tests passed. I also fixed the copyright lines. Regards, Ogata "Langer, Christoph" <[email protected]> wrote on 2017/06/28 21:04:36: > From: "Langer, Christoph" <[email protected]> > To: Kazunori Ogata <[email protected]> > Cc: Alan Bateman <[email protected]>, Claes Redestad > <[email protected]>, core-libs-dev <core-libs- > [email protected]>, "[email protected]" <nio- > [email protected]>, "[email protected]" <ppc-aix-port- > [email protected]> > Date: 2017/06/28 21:04 > Subject: RE: 8179527: Ineffective use of volatile hurts performance of > Charset.atBugLevel() > > Hi Ogata, > > > > remove the second run with -Dsun.nio.cs.bugLevel=1.4 > > > > How can I do this? Is it sufficient to remove the following line at the > > beginning of the file?: "@run main/othervm -Dsun.nio.cs.bugLevel=1.4 > > EmptyCharsetName" > > Yes, this line should be removed. Currently there are two @run directives > which cause running the testcase twice. Once in normal mode and once with > bugLevel set to 1.4. So, if "sun.nio.cs.bugLevel" ought to be removed then > the second iteration of the test is obsolete. And then one should probably > remove the whole "compat" handling in the test. > > Best regards > Christoph >
