Hi Alan,
yes the differences in location was an issue and the jdk8
CoreLibraries.gmk had a clash
regards
Mark
On 02/06/2014 14:37, Alan Bateman wrote:
On 31/05/2014 23:02, Mark Sheppard wrote:
Hi,
please oblige and review the following change
http://cr.openjdk.java.net/~msheppar/8041602/webrev/
which addresses the issue
https://bugs.openjdk.java.net/browse/JDK-8041602
which is a backport of
https://bugs.openjdk.java.net/browse/JDK-8035340
the original changeset didn't apply cleanly and was applied manually
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/740ffd98e35a
This looks okay to me, I'm curious as whether it was just the new
location in the jdk9/jdk tree that caused the issue or another change.
A minor comment on L48 where it could be "if (result .." to be
consistent.
-Alan
