On 06/10/10 13:51, Alan Bateman wrote:
Mandy Chung wrote:Looks fine to me. Minor comment is that you might want to rename INTERNAL_OPTIONS_FILENAME given that you've renamed the file name.Alan,Here is the revised webrev: http://cr.openjdk.java.net/~mchung/6959965/webrev.01/
Done
Thanks for expanding the test coverage for this.
Thanks for the review too. Mandy
