<Security Dev> [9] Review Request: 8028266 Tidy warnings cleanup for packages java.security/javax.security
alexander stepanov
alexander.v.stepanov at oracle.com
Tue Apr 22 13:41:55 UTC 2014
Hello Bradford,
Thanks,
Could you please review again:
http://cr.openjdk.java.net/~yan/JDK-8028266/webrev.01/
> Who is going to help you with that?
I didn't communicate with JCE team before. So could you please appoint
me someone to contact? Thanks.
Regards,
Alexander
On 22.04.2014 1:08, Bradford Wetmore wrote:
> We usually update the copyright dates:
>
> * Copyright (c) 1997, 2014, Oracle and/or its affiliates. All rights
> reserved.
>
> javax/security/auth/kerberos/package-info.java
> ==============================================
>
> "yes", or "no", case-insensitive.
> ->
> "yes", or "no", and values are case-insensitive.
>
> I noticed in several places that while your changes are ok, it doesn't
> update the <i>value</i> instead of switching to {@code value}.
>
> In a previous webrev, I mentioned that you'll need to work with
> someone to update the built JCE binaries for the javax.crypto
> changes. Who is going to help you with that?
>
> Brad
>
>
>
> On 4/21/2014 3:11 AM, alexander stepanov wrote:
>> Hello,
>>
>> Could you please review the fix for the following bug:
>> https://bugs.openjdk.java.net/browse/JDK-8028266
>>
>> Webrev corresponding:
>> http://cr.openjdk.java.net/~yan/JDK-8028266/webrev.00/
>>
>> Just a minor cleanup of javadoc to avoid tidy warnings; no other code
>> affected.
>>
>> Thanks.
>>
>> Regards,
>> Alexander
More information about the security-dev
mailing list