On 08/10/2015 10:47 PM, Weijun Wang wrote:
Everything is fine.

One thing in AuthPermission.java:
+ * {@link javax.security.auth.login.LoginContext}, and
+ * {@link javax.security.auth.login.Configuration} objects.

I remember you can add an additional label in {@link}, otherwise, the
text is too long.

It's not a problem in this case. By default, the link uses the name of the class for the link so it appears as "LoginContext and Configuration objects.".

--Sean


Thanks
Max

On 08/07/2015 04:06 AM, Sean Mullan wrote:
Please review this small specification change to allow implementations
to define additional implementation-specific target names for
SecurityPermission, RuntimePermission, and AuthPermission:

webrev: http://cr.openjdk.java.net/~mullan/webrevs/8077055/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8077055

Thanks,
Sean

Reply via email to