RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()

Bengt Rutisson bengt.rutisson at oracle.com
Thu Nov 19 09:04:06 UTC 2015



On 2015-11-19 10:00, Thomas Schatzl wrote:
> Hi Bengt,
>
> On Thu, 2015-11-19 at 09:24 +0100, Bengt Rutisson wrote:
>> Hi Thomas,
>>
>> Thanks for looking at this!
>>
>> On 2015-11-18 21:36, Thomas Schatzl wrote:
>>> Hi Bengt,
>>>
>>> On Wed, 2015-11-18 at 20:47 +0100, Bengt Rutisson wrote:
>>>> Hi everyone,
>>>>
>>>> Could I have a couple of reviews for this patch to remove some debugging
>>>> code?
>>>>
>>>> http://cr.openjdk.java.net/~brutisso/8143255/webrev.00/
>>>> https://bugs.openjdk.java.net/browse/JDK-8143255
>>>     afaik the memory_total parameter to the various methods is only every
>>> consumed by the now removed printout.
>>>
>>> Can you please remove that too?
>> Good point. Here's an updated webrev:
>> http://cr.openjdk.java.net/~brutisso/8143255/webrev.01/
>    looks good.

Thanks, Thomas!
Bengt

>
> Thomas
>
>



More information about the hotspot-runtime-dev mailing list