Re: Review request: JDK-8160398 (jdeps) Replace a list of JDK 8 internal API for detecting if it's removed in JDK 9 or later

[Daniel Fuchs]

Hi Mandy,

I have not finished reviewing yet - but as early feedback I believe
that the following probably has a mistake:

jdk.jdeps/share/classes/com/sun/tools/jdeps/Analyzer.java

 392
 393             if (!jdk8Internals.contains(pn)) {
 394                 return false;
 395             }
 396
 397             return jdk8Internals.contains(pn);

I suspect that one of the 'pn' should be 'cn' ?

best regards,

-- daniel

On 13/07/16 02:25, Mandy Chung wrote:

This patch includes a generated list of JDK 8 internal packages (a trimmed down 
list) such that jdeps can determine if any JDK internal API is referenced but 
removed in JDK 9 or later release.

http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8160398/webrev.00/

Mandy