Review request: module-info class file update

[Alan Bateman]

On 03/02/2012 20:52, Mandy Chung wrote:
> Jon and I are working together to update javac and runtime per the 
> latest language and class file specification for module declaration 
> [1].   Key changes include:
>
> 1. ModuleRequires attribute and ModuleProvides attribute
>     - use u4 flags and add SYNTHESIZED flag
> 2. new CONSTANT_ModuleQuery_info
> 3. exports <a package name>
> 4. default platform module is "jdk.base"
>     - will switch to "java.base" when the alias support is implemented
> 5. The jar tool synthesizes "requires jdk.jre" in the generated 
> module-info
>    that is the entire JRE.  If the legacy jar requires other modules, 
> it requires
>    to define its dependences properly.
>
> Webrev at:
>    http://cr.openjdk.java.net/~mchung/jigsaw/webrevs/jvms-update.00/
>
> Thanks
> Mandy
> [1] http://openjdk.java.net/projects/jigsaw/doc/lang-vm.html
>
> P.S. There is one javac failure we are currently diagnosing.  I will 
> send a separate review for that issue once we resolve it as I believe 
> it's fairly isolated one.
I went through the changes and they look fine to me. It's good to get 
the implementation sync'ed up.

-Alan