Request for review (XXS): 7171936: LOG_G incorrectly defined in globalDefinitions.hpp
Christian Thalinger
christian.thalinger at oracle.com
Tue May 29 16:32:02 UTC 2012
Looks good to me.
-- Chris
On May 25, 2012, at 1:47 PM, Bengt Rutisson <bengt.rutisson at oracle.com> wrote:
>
> Hi all,
>
> Can I have a couple of reviews for this really small change:
> http://cr.openjdk.java.net/~brutisso/7171936-LOG_G/webrev.00/
>
> Background:
> The constant LOG_G in globalDefinitions.hpp is incorrectly definied as 40. It should be 30. LOG_G is not being used anywhere, so rather than fixing it we should remove it.
>
> LOG_K is also not used and LOG_M is only used in one single place. I suggest that we remove LOG_K and move LOG_M to where it is being used.
>
> Thanks Ulf Zibis for catching the wrongly defined constant.
>
> Bengt
More information about the hotspot-gc-dev
mailing list