RFR: 8143229: Replace the develop level with develop macros in Unified Logging

Coleen Phillimore coleen.phillimore at oracle.com
Fri Nov 20 22:15:56 UTC 2015



On 11/20/15 4:58 PM, Coleen Phillimore wrote:
>
> Hi Marcus,
>
> This looks good, except I have one question:
> http://cr.openjdk.java.net/~mlarsson/8143229/webrev.00/src/share/vm/logging/logConfiguration.cpp.udiff.html 
>
>
> Shouldn't this be log_develop_info(logging) not trace?  Why did you 
> pick trace?

Aside: if you change it, I don't need to see another version.
Coleen
>
> thanks,
> Coleen
>
> On 11/20/15 6:58 AM, Marcus Larsson wrote:
>> Hi,
>>
>> Please review the following patch to replace the previous 'develop' 
>> level with macros, in the unified logging framework.
>> The macros are essentially regular log calls inside a #ifndef 
>> PRODUCT. In the product they do nothing / return false.
>> This allows the regular log levels to be used for develop-only logging.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~mlarsson/8143229/webrev.00/
>>
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8143229
>>
>> Thanks,
>> Marcus
>



More information about the hotspot-runtime-dev mailing list