Please, review fix for:
  https://bugs.openjdk.java.net/browse/JDK-8222934

Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8222934-jvmti-depr-option.1/


Summary:
  David, in review for https://bugs.openjdk.java.net/browse/JDK-8222934 suggested:    1. I would have suggested to add "(Deprecated)" to the description of the new flag in globals.hpp    2. The new flag should have been added to the deprecated VM options tests.    3. The new test should run in both a positive and negative mode so that it also checks that the new flag works.

  The webrev above implements this suggestion.


Testing:
  In progress: Submit mach5 run for the updated tests.


Thanks,
Serguei




Reply via email to