> On Jun 18, 2017, at 8:33 PM, Weijun Wang <weijun.w...@oracle.com> wrote:
>
> Hi All
>
> Please take a review at
>
> http://cr.openjdk.java.net/~weijun/8182118/webrev.00/
>
> Basically, a description line is added into package-info.java of each of
> these packages:
>
> - com/sun/security/auth:
>
> Contains the implementation of {@code java.security.Principal}.
>
> - com/sun/security/auth/module:
>
> Contains the implementation of {@code javax.security.auth.spi.LoginModule}.
>
> - com/sun/security/auth/login:
>
> Contains the implementation of {@code
> javax.security.auth.login.Configuration}.
>
> - com/sun/security/auth/callback:
>
> Contains the implementation of {@code javax.security.auth.callback.Callback}.
>
What about “Provides the implementation of ….”
I suggest to use @link to the type.
> with @since 1.4.
>
> I thought about using {@link java.security.Principal} but seems it's not
> supported in package-info.java.
java/lang/package-info.java and many package summary use @link.
>
> BTW, is this bug meant for JDK 9? I just read the mail from Mark saying only
> P1 fixes will be allowed from now on.
If you push it your Monday, you should be able to make jdk-9+175 integration
(6/22 GAC). Otherwise P1 fixes only.
Mandy
