And it would be good to use the GetIntegerAction with the default and get rid of the if clause.
With best regards. Petr. On 04.04.2014, at 19:09, Dmitriy Ermashov <[email protected]> wrote: > Oops, small typo! > Will fix without webrev, ok? > > With regards, > Dmitriy Ermashov > > 04 апр. 2014 г., в 19:06, Sergey Bylokhov <[email protected]> > написал(а): > >> On 4/4/14 7:02 PM, Dmitriy Ermashov wrote: >>> Hi, >>> Please, review the changeset for: >>> https://bugs.openjdk.java.net/browse/JDK-8038631 >>> >>> Webrev is here: >>> http://cr.openjdk.java.net/~yan/8038631/webrev.05/ >>> >>> Last changes consist of: >>> 1. Modified property read process, added >>> sun.security.action.GetIntegerAction usage. >>> 2. Javadoc improvements, added @link, rephrased some sentences. >>> 3. Refactored overloaded glide() methods. >> Nice refactoring "GetIntegerAction1". >> >> >> -- >> Best regards, Sergey.
