Review Request JDK-8171323: generate dot file for java.se and java.se.ee with only API dependences
Mandy Chung
mandy.chung at oracle.com
Fri Dec 16 01:27:02 UTC 2016
> On Dec 15, 2016, at 5:12 PM, Paul Sandoz <paul.sandoz at oracle.com> wrote:
>
>
>> On 15 Dec 2016, at 10:53, Mandy Chung <mandy.chung at oracle.com> wrote:
>>
>> This updates the GenGraphs build tool to generate a dot file for
>> `java.se` and `java.se.ee` modules including API dependences only
>> that can be used to display Java SE module graph.
>>
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171323/webrev.00/
>>
>
> Looks ok.
>
> You can marginally improve genDotFile:
>
>
> List<ModuleDescriptor> mds = cf.modules().stream()
> .map(ResolvedModule::reference)
> .map(ModuleReference::descriptor)
> .collect(toList());
>
> …
>
> printGraph(out, name, gengraph(cf), mds.stream().collect(toMap(ModuleDescriptor::name, Function.identity()));
>
> …
>
> printGraph(out, name, graph,
> mds.stream()
> .filter(md -> !md.name().startsWith("jdk.”) && graph.nodes().contains(md.name()))
> .collect(toMap(ModuleDescriptor::name, Function.identity()))
Sure I can clean that up. Thanks.
Mandy
More information about the jigsaw-dev
mailing list