RFR (S): 8079091: Remove dictionary NULL check on common path of BlockFreeList methods

Kim Barrett kim.barrett at oracle.com
Fri May 1 00:31:27 UTC 2015


On Apr 30, 2015, at 4:24 PM, Jon Masamitsu <jon.masamitsu at oracle.com> wrote:
> 
> 
> 
> On 4/30/2015 12:15 PM, Kim Barrett wrote:
>> On Apr 30, 2015, at 3:37 AM, Jungwoo Ha <jwha at google.com> wrote:
>>> My original change (not embedding CHeapObj): http://cr.openjdk.java.net/~jwha/8079091/webrev.00/
>>> Jon's suggestion (from hotspot-gc-dev): http://cr.openjdk.java.net/~jwha/8079091/webrev.02/
>>> 
>>> Please take a look on .02.
>> Sorry, but version .02 gets a No from me.
> 
> All,
> 
> Let's go back to Jungwoo's 00 version.  It's a clean, limited change.
> 
> Objections?

That’s mostly ok with me.

I think the constructor and destructor suggestions that Stefan and I made should be added.



More information about the hotspot-runtime-dev mailing list