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 daniel.fuchs at oracle.com
Wed Jul 13 08:55:08 UTC 2016


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
>



More information about the core-libs-dev mailing list