jigsaw now sync'ed with jdk8-b23

[Alan Bateman]

On 31/01/2012 23:49, Mandy Chung wrote:
>
> This depends on the services support that Alan is working on [1].   
> The service loader is not yet module-aware.  Prior to the module views 
> and exports change, all public types including 
> sun.java2d.jules.JulesRenderingEngine are exported and that's why it 
> was working previously.
>
> sun.java2d.pipe.RenderingEngine calls ServiceLoader.loadInstalled 
> method to create a new service loader using the extension class 
> loader.  In module mode, there isn't any extension class loader but 
> instead it should use the caller's module loader.
>
> Alan, what do you think to integrate the change you have in the 
> ServiceLoader class that uses the proper module loader in module mode 
> first?   I think that can be separated from the services work.
I re-based the services patch after the views+exports work was pushed 
and just checked that it has the right declarations:

$ jmod ls -v
:
jdk.desktop at 8-ea
   :
   requires service sun.java2d.pipe.RenderingEngine
   :
   provides service sun.java2d.pipe.RenderingEngine with 
sun.java2d.jules.JulesRenderingEngine
   provides service sun.java2d.pipe.RenderingEngine with 
sun.java2d.pisces.PiscesRenderingEngine

I don't think the ServiceLoader changes on their own will fix this as it 
requires the services changes in the runtime. As many things are 
currently broken then I think I should just get the first version in. I 
have a second patch to it in the works that re-works the Resolver code 
to handle services dependencies properly (the first version was to get 
things working).

-Alan