Core Libs Dev[9] Review Request for 8030709: Tidy warnings cleanup for java.lang package

2014-04-15 Thread alexander stepanov
Sorry, the webrev was updated to include one other minor change in javax/script/ScriptEngineFactory.java to avoid creation of separate review request: http://cr.openjdk.java.net/~yan/8030709/webrev.01/ Regards, Alexander On 10.04.2014 12:24, alexander stepanov wrote: BigInteger.java from

Re: Core Libs Dev[9] Review Request for 8030709: Tidy warnings cleanup for java.lang package

2014-04-15 Thread Lance Andersen
looks ok On Apr 15, 2014, at 9:11 AM, alexander stepanov alexander.v.stepa...@oracle.com wrote: Sorry, the webrev was updated to include one other minor change in javax/script/ScriptEngineFactory.java to avoid creation of separate review request:

Re: Core Libs Dev[9] Review Request for 8030709: Tidy warnings cleanup for java.lang package

2014-04-15 Thread alexander stepanov
thanks! On 15.04.2014 17:24, Lance Andersen wrote: looks ok On Apr 15, 2014, at 9:11 AM, alexander stepanov alexander.v.stepa...@oracle.com mailto:alexander.v.stepa...@oracle.com wrote: Sorry, the webrev was updated to include one other minor change in javax/script/ScriptEngineFactory.java

Core Libs Dev[9] Review Request for 8030709: Tidy warnings cleanup for java.lang package

2014-04-10 Thread alexander stepanov
Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8030709 Webrev corresponding: http://cr.openjdk.java.net/~yan/8030709/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. BigInteger.java from

Core Libs Dev[9] Review Request for 8030709: Tidy warnings cleanup for java.lang package

2014-04-10 Thread alexander stepanov
BigInteger.java from java.lang was also touched. sorry, from java.math On 10.04.2014 12:23, alexander stepanov wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8030709 Webrev corresponding: