Xueming Shen wrote:
Kelly,Alan might be off-line now. Can you help review the change? The change is to define our own MIN2. http://cr.openjdk.java.net/~sherman/6990639/webrev/Verified via jprt hotspotwest just know. There was a failure on windows-x64, but it does not appear to be related. Alan said it might be 2D (lcms) related.
Looks fine to me.The lcms build failure is fixed in the 2d repo [1] but it's in tl/jdk7/jdk yet.
-Alan [1] http://hg.openjdk.java.net/jdk7/2d/jdk/rev/160f7ffc3d14
