Code Review Request for 6953295 and related changes to add keytool to jdk.tools.base
david.holmes at oracle.com
Thu Oct 6 10:21:30 PDT 2011
On 7/10/2011 2:12 AM, Sean Mullan wrote:
> On 10/5/11 10:19 PM, David Holmes wrote:
>> There seem to be a lot of Java code changes in that webrev that seem
>> unrelated to moving keytool ???
> Not really. There is a bit of extra cleanup/dead-code removal in CR
> 6953295 like removing sun.security.util.BigInt but everything else is
> necessary to move keytool and jarsigner to jdk.tools.base. Basically a
> bunch of classes that only jarsigner/keytool use were moved to different
Ok. It wasn't clear to me that a lot of the changes (eg interface ->
abstract class) were needed to assist the package changes. Plus there
was the use of diamond, try-with-resources etc.
>> On 6/10/2011 11:24 AM, Mandy Chung wrote:
>>> On 9/20/11 2:44 PM, Sean Mullan wrote:
>>>> New webrev:
>>> Looks good.
>>> BTW, while I'm merging the jdk8 repos with the jigsaw repos, I realize
>>> that the restructuring in the timestamping code you did back in April
>>> [1, 2] is also orthogonal to jigsaw. I think it'd be best to port that
>>> change to JDK 8 repo as other jdk8 changes are happening there and it
>>> will also help the merge. Would you be able to push that non-module
>>> related change to jdk8 as well?
>>>  http://hg.openjdk.java.net/jigsaw/jigsaw/jdk/rev/82c28137a816
More information about the jigsaw-dev