The 3 crypto files look fine to me.
BTW I did notice that you preserve the Copyright range years in all but 
src/java.base/share/classes/javax/crypto/package-info.java

Thanks.


> On 22 May 2015, at 17:42, joe darcy <[email protected]> wrote:
> 
> Hello security library team,
> 
> I neglected to include you in the original request for review of this change. 
> I'd like to replace all the package.html files in the java.base module with 
> package-info.java files and a few cryptography packages are involved; please 
> review.
> 
> Thanks,
> 
> -Joe
> 
> 
> -------- Forwarded Message --------
> Subject:      JDK 9 RFR of JDK-8080901: Replace package.html files with 
> package-info.java in the java.base module
> Date: Thu, 21 May 2015 15:57:36 -0700
> From: joe darcy <[email protected]> <mailto:[email protected]>
> Organization: Oracle Corporation
> To:   core-libs-dev <[email protected]> 
> <mailto:[email protected]>
> 
> Hello,
> 
> Please review these changes to replace old-style package.html files with 
> package-info.java files in the base module:
> 
>     JDK-8080901: Replace package.html files with package-info.java in 
> the java.base module
>     http://cr.openjdk.java.net/~darcy/8080901.0/ 
> <http://cr.openjdk.java.net/~darcy/8080901.0/>
> 
> (The package.html file for src/java.base/share/classes/sun/reflect is 
> just removed rather than replaced since it contains "new" porting 
> information that was current as of 2001.)
> 
> Thanks,
> 
> -Joe
> 
> 

Reply via email to