RFR (S): 8024662: gc/arguments/TestUseCompressedOopsErgo.java does not compile
stefan.karlsson at oracle.com
Tue Sep 17 05:54:56 PDT 2013
On 09/17/2013 02:38 PM, Thomas Schatzl wrote:
> can I have reviews for the following small change? It fixes
> compilation and running of the above JTREG test.
> The compilation issue has been slipped through my testing, and the
> problem at execution is actually caused by another change pushed just
> before this one that changed the name of the "ClassMetaspaceSize" global
> to "MetaspaceSize".
ClassMetaspaceSize was renamed to CompressedClassSpaceSize, so this
change seems incorrect.
> I also started to refactor just a little, i.e. replace one manual check
> for equality by calls to the Asserts class that has been introduced in
> another CR at around the same time I think.
> More refactoring will be done in JDK-8024925, an issue I found while
> improving the tests for this change (and JDK-8024669). Fixing
> JDK-8024925 seems to be out of scope here.
> Please consider reviewing in conjunction with the RFR for JDK-8024669
> which I will send too.
> jprt, manual jtreg execution, jprt jtreg testing
More information about the hotspot-gc-dev