On 4/2/15 12:58 PM, Shanliang Jiang wrote:
Hi,
I have to ask the review again because I need to modify:
langtools/src/jdk.dev/share/classes/com/sun/tools/jdeps/Profile.java
The issue was found when langtools tests were added into my test list.
The new version is:
http://cr.openjdk.java.net/~sjiang/JDK-8042901/02/
Thanks for catching the jdeps change. Profile.java change looks okay
and this list is temporary. jdeps will be updated to determine the
exported APIs at runtime when the module system is moving along.
which integrated also the Mandy's comments in the following mail.
CheckSomeMXBeanImplPackage.java
line 45-50 & 58-60: should be called unconditionally since they
should pass if java.management is present.
Okay to push once the test is updated.
Mandy