[9] RFR(S): CodeHeap::next_free should be renamed

Tobias Hartmann tobias.hartmann at oracle.com
Fri Jan 9 10:17:24 UTC 2015


Hi Zoltan,

that looks good (not a reviewer). I like the comment!

Thanks,
Tobias

On 09.01.2015 11:10, Zoltán Majó wrote:
> Hi,
> 
> 
> please review the following small patch.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8065894
> 
> Problem: The method 'CodeHeap::next_free' does not return the next free
> HeapBlock as the name and the comment suggest. Actually, it returns the
> following block that is _not_ free, i.e., the next used block.
> 
> Solution: The name of the method and the comment about the method is changed
> accordingly.
> 
> Webrev: http://cr.openjdk.java.net/~zmajo/8065894/webrev.00/
> 
> Testing: JPRT.
> 
> Thank you and best regards,
> 
> 
> Zoltan
> 


More information about the hotspot-compiler-dev mailing list