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.