Looks good to me.
BTW, I thought that webrev would effectively extract the diffs even when
files were moved around.
Naoto
On 1/10/13 9:49 AM, Brent Christian wrote:
Hi,
The test case for 8003228 fails in certain environments. Also the
version that was pushed was missing a couple small changes.
The code changes to fix these issues are here:
http://cr.openjdk.java.net/~bchristi/8005962/webrev.00/
The test case will also be moved from java/util/Properties/ to
java/lang/System/. Webrev wouldn't do well showing that part of the
change, so it's not reflected there. Instead I generated a webrev to
better show the code changes.
Thanks,
-Brent