Hi,

please review this rewrite/cleanup of the MakeClasslist tool to operate on the output of -XX:DumpLoadedClassList rather than -XX:+TraceClassLoadingPreorder. Since the tool became rather trivial I opted to write it in nashorn-compliant javascript to streamline
the usage.

Bug: https://bugs.openjdk.java.net/browse/JDK-8081589
Webrev: http://cr.openjdk.java.net/~redestad/8081589/webrev.02

A number of undocumented/unused tests were removed and an outdated README was incorporated into the tool source itself, among other things clarifying that the checksum needs to be calculated and added to the classlist before checking it into the workspace.

I've asked around about how to go about adding tests for standalone tools like these, but didn't come up with a good answer. If someone insists I add a small test to this I'd hope
there's some insight into how best to do that (shell-based jtreg test?)

Thanks!

/Claes

Reply via email to