RFR: 8028357: Unnecessary allocation in AliasFileParser

Jaroslav Bachorik jaroslav.bachorik at oracle.com
Tue Dec 30 13:43:34 UTC 2014


On 29.12.2014 16:51, Claes Redestad wrote:
> Hi,
>
> some classes in jvmstat/perfdata code contains debug logging code
> predating the logging APIs, which
> provokes some unnecessary allocation in certain applications.
>
> Since the debug logging can't have been used for quite some time, I
> propose to remove it altogether
> rather than converting it to the j.u.l. framework.

While it's good to see the dead code gone I would prefer keeping the 
logging code just commented out - the logging code could be useful when 
investigating any future failures.

Converting to j.u.l. wouldn't probably completely remove the unnecessary 
allocation (due to object arrays for varargs) unless you guard each log 
call for the required logging level.

-JB-

>
> bug: https://bugs.openjdk.java.net/browse/JDK-8028357
> webrev: http://cr.openjdk.java.net/~redestad/8028357/webrev.01/
>
> /Claes



More information about the serviceability-dev mailing list