hg: jdk8/tl/jdk: 7068662: Reserve and restore the default locale

2011-07-29 Thread xuelei . fan
Changeset: cea7c749f805 Author:xuelei Date: 2011-07-29 02:50 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cea7c749f805 7068662: Reserve and restore the default locale Reviewed-by: alanb, weijun ! test/com/sun/org/apache/xml/internal/security/exceptions/LocaleTest.java !

Re: ArrayList.subList

2011-07-29 Thread Doug Lea
On 07/28/11 19:20, Doug Lea wrote: Thanks. I hadn't noticed that the parenthesized i.e., this list in the List specs (http://download.oracle.com/javase/7/docs/api/java/util/List.html#subList%28int,%20int%29 -- pasted below) overly constrains the interpretation of backing list, which would

Code review request: 7072523 java.math should be built with javac -Xlint:all -Werror

2011-07-29 Thread Alexandre Boulgakov
Hello Joe, Could you review the attached two-line update to ensure that no javac warnings are introduced into java.math? Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072523 Thanks, Sasha diff -r 7525866a4046 make/java/math/Makefile --- a/make/java/math/Makefile Thu Jul 28

Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror

2011-07-29 Thread Alexandre Boulgakov
Hello, Please review these JNDI changes. Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072353http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7068616 webrev: http://cr.openjdk.java.net/~mduigou/7072353/0/webrev/ Thanks, Sasha

Re: Code review request: 7072523 java.math should be built with javac -Xlint:all -Werror

2011-07-29 Thread Joe Darcy
On 7/29/2011 4:32 PM, Alexandre Boulgakov wrote: Hello Joe, Could you review the attached two-line update to ensure that no javac warnings are introduced into java.math? Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072523 Thanks, Sasha Approved. Thanks, -Joe

hg: jdk8/tl/jdk: 7072523: java.math should be built with javac -Xlint:all -Werror

2011-07-29 Thread jonathan . gibbons
Changeset: 4030297803eb Author:jjg Date: 2011-07-29 16:45 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4030297803eb 7072523: java.math should be built with javac -Xlint:all -Werror Reviewed-by: darcy Contributed-by: alexandre.boulga...@oracle.com ! make/java/math/Makefile