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


Reply via email to