Re: [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 > wrote: Sorry, the webrev was updated to include one other minor change in javax/script/ScriptEngineFactory.java to avoid creation of separate

Re: [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 wrote: > 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, > Alex

[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 ja

[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: http://cr.openjdk.java.net/~yan/8030709/w

[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 java.la