RFR: 8081007: Remove redundant active worker variables and calls in ParNewGeneration::collect
Stefan Karlsson
stefan.karlsson at oracle.com
Tue May 26 08:04:28 UTC 2015
On 2015-05-26 10:03, Erik Helin wrote:
> On 2015-05-25, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this small patch to unify how the number of active workers are
>> passed around in ParNewGeneration::collect.
>>
>> http://cr.openjdk.java.net/~stefank/8081007/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8081007
> Looks good, Reviewed.
Thanks, Erik.
StefanK
>
> Thanks,
> Erik
>
>> - Removed the unnecessary n_workers variable.
>> - Replaced workers()->active_workers() with active_workers.
>> - Removed workers NULL check assert, since the exact same assert is
>> performed earlier in the function.
>>
>> Thanks,
>> StefanK
More information about the hotspot-gc-dev
mailing list