On 08/12/2016 14:41, Remi Forax wrote:

Chris,
this patch is rather ugly.
Patching the ModuleDescriptor is not a good idea, maybe it's means that we need 
a system of metadata associated with a ModuleDescriptor,
the hash of a module (which is not part of the module spec but the OpenJDK 
implementation) can be also considered as metadata.

There is refactoring in flight that drops the JDK-specific meta data from ModuleDescriptor. Once all the pieces are merged together then it should look a lot better.

-Alan.

Reply via email to