[9] RFR(S): 8035946: Use ResourceHashtable for dependency checking
Vladimir Kozlov
vladimir.kozlov at oracle.com
Fri Feb 28 09:35:26 PST 2014
Albert,
This looks good. Thank you for doing it.
Can you fix indent of second line of DepTable typedef to start it under after opening "<"?
Thanks,
Vladimir
On 2/28/14 6:58 AM, Albert wrote:
> Hi,
>
> I send this to hotspot-dev, since the change contains a small modification in
> resourceHash.hpp
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8035946
>
> Problem:
> JDK-8034839 introduced a new hashtable for caching checked dependencies. However, a hashtable that already existed prior
> to JDK-8034839 (ResourceHashtable) could have been used instead of introducing the new one.
>
> Solution:
> Use ResourceHashtable and remove the new hashtable that was introduced with JDK-8034839.
>
> Testing:
> jprt, failing test
>
> webrev:
> http://cr.openjdk.java.net/~anoll/8035946/webrev.00/
>
> Many thanks in advance,
> Albert
More information about the hotspot-dev
mailing list