RFR(s) 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()

David Holmes david.holmes at oracle.com
Thu Jan 21 08:24:05 UTC 2016


Looks good to me!

Thanks,
David

On 21/01/2016 6:02 PM, Per Liden wrote:
> Hi,
>
> All logging macros exposed for public consumption by the Unified logging
> framework follows the naming convention "log_*", except for
> develop_log_is_enabled(). This patch renames that macro to
> log_develop_is_enabled(). With such a rename we will have:
>
> log_trace
> log_debug
> log_info
> log_warning
> log_error
> log_is_enabled
>
> followed by the develop versions:
>
> log_develop_trace
> log_develop_debug
> log_develop_info
> log_develop_warning
> log_develop_error
> log_develop_is_enabled (instead of develop_log_is_enabled)
>
>
> Webrev: http://cr.openjdk.java.net/~pliden/8147918/webrev.0/
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8147918
>
> cheers,
> Per


More information about the hotspot-dev mailing list