Thank you, Christoph. Regards, Ogata
"Langer, Christoph" <[email protected]> wrote on 2017/06/28 15:45:41: > 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 15:45 > Subject: RE: 8179527: Ineffective use of volatile hurts performance of > Charset.atBugLevel() > > Hi Ogata, > > this looks fine. > > The copyright year in Charset-X-Coder.java.template needs to be updated, > but I can do that when I push it. > > Waiting for another review... > > Best regards > Christoph > > > -----Original Message----- > > From: Kazunori Ogata [mailto:[email protected]] > > Sent: Mittwoch, 28. Juni 2017 08:21 > > To: Langer, Christoph <[email protected]> > > Cc: Alan Bateman <[email protected]>; Claes Redestad > > <[email protected]>; core-libs-dev <core-libs- > > [email protected]>; [email protected]; ppc-aix-port- > > [email protected] > > Subject: RE: 8179527: Ineffective use of volatile hurts performance of > > Charset.atBugLevel() > > > > Hi Christoph, > > > > Thank you for your suggestions and offering to sponsor my changes. > > > > Here is the updated webrev that removes the atBugLevel() definition in > > Charset.java and its call sites in Charset.java and > > Charset-X-Coder.java.template. Please review this: > > > > http://cr.openjdk.java.net/~horii/8179527/webrev.01/ > > > > > > Regards, > > Ogata > > > > > > "Langer, Christoph" <[email protected]> wrote on 2017/06/28 > > 03:32:51: > > > > > From: "Langer, Christoph" <[email protected]> > > > To: Alan Bateman <[email protected]>, Kazunori Ogata > > <[email protected]> > > > Cc: "[email protected]" <ppc-aix-port- > > > [email protected]>, Claes Redestad <[email protected]>, > > core- > > > libs-dev <[email protected]>, "[email protected]" > > > <[email protected]> > > > Date: 2017/06/28 03:32 > > > Subject: RE: 8179527: Ineffective use of volatile hurts performance of > > > Charset.atBugLevel() > > > > > > Hi Ogata, > > > > > > I think I agree with Alan that the Charset.atBugLevel() method can > > > completely be eliminated from java/nio/charset. > > > > > > Ogata, would you respin your change to remove it and post it for review? > > I > > > can then sponsor it for you. > > > > > > @Alan: Do we need a CSR ("Compatibility & Specification Review") request > > > > > here since support for "sun.nio.cs.bugLevel" will be removed? > > > > > > Best regards > > > Christoph > > > > > > > -----Original Message----- > > > > From: Alan Bateman [mailto:[email protected]] > > > > Sent: Dienstag, 27. Juni 2017 10:13 > > > > To: Claes Redestad <[email protected]>; Langer, Christoph > > > > <[email protected]>; Kazunori Ogata <[email protected]>; > > > > core-libs-dev <[email protected]>; > > [email protected] > > > > Cc: [email protected] > > > > Subject: Re: 8179527: Ineffective use of volatile hurts performance of > > > > Charset.atBugLevel() > > > > > > > > On 27/06/2017 08:36, Claes Redestad wrote: > > > > > > > > > > The check of Charset.atBugLevel in checkName should no longer > > happen > > > > > for the majority of situations, as that test is now only done if the > > > > > charset name is "" (see > > > > > https://bugs.openjdk.java.net/browse/JDK-8174831), > > > > Kazunori's mail didn't mention the JDK build he is using, it may have > > > > been JDK 8 rather than JDK 9. > > > > > > > > > since what differs between 1.4 and 1.5 was apparently whether or not > > > > > the empty string was to be accepted as a valid Charset... > > > > > > > > > > So yes, if we can get rid of the test altogether, we'll be even > > better > > > > > off! > > > > JDK-4786884 is the original issue. If there was any code dependent on > > > > the broken behavior in 1.4 then I would expect it should have been > > fixed > > > > by now. So I think it can be removed. > > > > > > > > -Alan > > >
