On 3/30/18 7:38 PM, Peter Levart wrote:


On 03/30/2018 01:31 PM, mandy chung wrote:


On 3/30/18 6:53 PM, Alan Bateman wrote:


8200134: Improve ModuleHashesBuilder
http://cr.openjdk.java.net/~martin/webrevs/jdk/ModuleHashesBuilder/
https://bugs.openjdk.java.net/browse/JDK-8200134
I suspect Mandy will want to look at this one. We also need to decide if there is a better place for it as it is only used by tools (`jar`, `jmod`, ...), it is not part of the runtime support.

It's in java.base to avoid duplicating this class in both jdk.jar and jdk.jlink module.  It's not a small method.  It's not part of the runtime support.  I can't think of any better place.

New module jdk.tools.internal ?

I considered it while I fear that this would become a dumping ground for tools utility although we start with one class.  The name would be jdk.internal.xxx.

Mandy

Reply via email to