RFR (XXS): JDK-8049327: [TESTBUG] gc/logging/TestGCId.java assumes default PrintGCID value is true

Bengt Rutisson bengt.rutisson at oracle.com
Mon Jul 7 12:52:16 UTC 2014

On 2014-07-07 10:45, Erik Helin wrote:
> On Monday 07 July 2014 10:02:37 AM Bengt Rutisson wrote:
>> Hi everyone,
>> Can I have a couple of reviews for this very small fix?
>> http://cr.openjdk.java.net/~brutisso/8049327/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8049327
> Looks good, Reviewed.

Thanks, Erik.

Got an off-line review from Thomas. All set to push this now.


> Thanks,
> Erik
>> When I backported the PrintGCID fix from JDK 9 to the 8u repo I changed
>> the default value to false for this flag. But I forgot to update the
>> test. The proposed patch simply changes the expected default behavior to
>> be false for PrintGCID.
>> Thanks,
>> Bengt

More information about the hotspot-gc-dev mailing list