On 24/10/2013 14:11, Fredrik Arvidsson wrote:
Hi
Please help me review the following small change:
Bug: https://bugs.openjdk.java.net/browse/JDK-8025576
Webrev:
http://cr.openjdk.java.net/~farvidsson/8025576/webrev.00/index.html
<http://cr.openjdk.java.net/%7Efarvidsson/8025576/webrev.00/index.html>
/Fredrik
This looks okay to me, the change seems to be hotspot-rt. However, where
you are planning to push this? If you push it to hotspot-rt/jdk then
it's okay, it's in the same forest as the fix. On the other hand, if you
push it to jdk8/tl/jdk then I assume we will have test failures until
the hotspot fix gets into master and down to jdk8/tl.
-Alan