Re: RFR: 8012261: update policytool to support java.net.HttpURLPermission

2013-05-17 Thread Sean Mullan
Looks fine to me. --Sean On 05/16/2013 10:17 PM, Weijun Wang wrote: Hi All Please take a look at http://cr.openjdk.java.net/~weijun/8012261/webrev.00/ which supports the new HttpURLPermission type introduced in 8010464: Evolve java networking same origin policy http://hg.open

hg: jdk8/tl/jdk: 2 new changesets

2013-05-17 Thread chris . hegarty
Changeset: 3981ad7ec458 Author:chegar Date: 2013-05-17 15:00 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3981ad7ec458 8014791: More ProblemList.txt updates (5/2013) Reviewed-by: alanb ! test/ProblemList.txt Changeset: fab0e4b682e8 Author:chegar Date: 2013-05-17

hg: jdk8/tl/jdk: 2 new changesets

2013-05-17 Thread chris . hegarty
Changeset: 222da3d4692a Author:chegar Date: 2013-05-17 16:44 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/222da3d4692a 8014783: java/net/HttpURLPermission/HttpURLPermissionTest.java leaves files open Reviewed-by: michaelm ! test/java/net/HttpURLPermission/HttpURLPermission

hg: jdk8/tl/jdk: 8011136: FileInputStream.available and skip inconsistencies

2013-05-17 Thread dan . xu
Changeset: 3b1450ee2bb9 Author:dxu Date: 2013-05-17 12:04 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3b1450ee2bb9 8011136: FileInputStream.available and skip inconsistencies Summary: Correct the behavior of available() and update related java specs for available() and sk

hg: jdk8/tl/langtools: 6885876: add comments to javac/util/Convert.java

2013-05-17 Thread jonathan . gibbons
Changeset: 0928f2cfbf8e Author:jjg Date: 2013-05-17 13:48 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/0928f2cfbf8e 6885876: add comments to javac/util/Convert.java Reviewed-by: mduigou ! src/share/classes/com/sun/tools/javac/util/Convert.java

Re: RFR: 8012261: update policytool to support java.net.HttpURLPermission

2013-05-17 Thread Christophe Ravel
Hi Max, What is your plan for regression tests for this change ? Regards, Christophe. Weijun Wang May 16, 2013 7:17 PM Hi All Please take a look at     http://cr.openjdk.java.net/~weijun/8012261/webrev.00/ which supports the new HttpURLPermission type introduced in   

hg: jdk8/tl/jaxp: 8013900: More warnings compiling jaxp.

2013-05-17 Thread daniel . fuchs
Changeset: 6443f5627744 Author:dfuchs Date: 2013-05-17 10:40 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/6443f5627744 8013900: More warnings compiling jaxp. Summary: Some internal implementation classes in Jaxp were redefining equals() without redefining hashCode(). This

hg: jdk8/tl/jdk: 8013900: More warnings compiling jaxp.

2013-05-17 Thread daniel . fuchs
Changeset: 68209420aac2 Author:dfuchs Date: 2013-05-17 10:40 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/68209420aac2 8013900: More warnings compiling jaxp. Summary: Some internal implementation classes in Jaxp were redefining equals() without redefining hashCode(). This

Re: RFR: 8012261: update policytool to support java.net.HttpURLPermission

2013-05-17 Thread Weijun Wang
Hi Christophe No regression test. If there is an existing manual test checking all permissions supported, I'll be glad to update the description to cover this new type. Thanks Max On 5/18/13 5:01 AM, Christophe Ravel wrote: Hi Max, What is your plan for regression tests for this change ?

hg: jdk8/tl/jdk: 8012261: update policytool to support java.net.HttpURLPermission

2013-05-17 Thread weijun . wang
Changeset: 0f7aaabed25f Author:weijun Date: 2013-05-18 10:15 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0f7aaabed25f 8012261: update policytool to support java.net.HttpURLPermission Reviewed-by: mullan ! src/share/classes/sun/security/tools/policytool/PolicyTool.java ! s