RFR: JDK-8151605: Change warning() to log_warning(gc) in the GC code

Bengt Rutisson bengt.rutisson at oracle.com
Fri Mar 11 10:35:56 UTC 2016


Hi Jon,

On 2016-03-10 18:53, Jon Masamitsu wrote:
> Need copyright updates
>
> src/share/vm/gc/shared/cardTableRS.cpp
> src/share/vm/gc/shared/generation.hpp
> src/share/vm/gc/shared/vmGCOperations.cpp
> src/share/vm/gc/cms/concurrentMarkSweepThread.cpp
> src/share/vm/gc/g1/g1CollectedHeap.cpp
> src/share/vm/gc/parallel/gcTaskThread.cpp
> || src/share/vm/gc/shared/cardGeneration.cpp

Fixed.

>
> Otherwise, look good.

Thanks for the review!

Bengt

>
> Jon
> **
> **
> On 03/10/2016 02:00 AM, Bengt Rutisson wrote:
>>
>> Hi everyone,
>>
>> Could I have a couple of reviews for this change?
>>
>> https://bugs.openjdk.java.net/browse/JDK-8151605
>> http://cr.openjdk.java.net/~brutisso/8151605/webrev.00/
>>
>> With the new logging framework we should use log_warning() instead of 
>> warning().
>>
>> Thanks,
>> Bengt
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20160311/cc04ea53/attachment.htm>


More information about the hotspot-gc-dev mailing list