RFR: 8081007: Remove redundant active worker variables and calls in ParNewGeneration::collect

Erik Helin erik.helin at oracle.com
Tue May 26 08:03:51 UTC 2015


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

> - 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