JDK-8151514: Remove the noisy NOISY debugging code from parCarTableModRefBS.cpp
bengt.rutisson at oracle.com
Thu Mar 10 07:55:13 UTC 2016
On 2016-03-09 14:16, Per Liden wrote:
> Looks good.
> On 2016-03-09 13:54, Bengt Rutisson wrote:
>> Hi everyone,
>> Could I have a couple of reviews for this change?
>> In CardTableModRefBSForCTRS::process_chunk_boundaries() there is a NOISY
>> definition to log some debug messages.
>> Several problems with this macro:
>> * The code inside process_chunk_boundaries() is very hard to read due to
>> all the NOISY lines.
>> * The logging is done to tty and not the logging framework-.
>> * At some point someone detected a conflict with a preprocessor flag.
>> It is better to remove this macro and just add relevant logging if we
>> every find the need for it.
More information about the hotspot-gc-dev