On 06/10/10 13:51, Alan Bateman wrote:
Mandy Chung wrote:
Alan,

Here is the revised webrev:
 http://cr.openjdk.java.net/~mchung/6959965/webrev.01/
Looks fine to me. Minor comment is that you might want to rename INTERNAL_OPTIONS_FILENAME given that you've renamed the file name.

Done

Thanks for expanding the test coverage for this.

Thanks for the review too.

Mandy

Reply via email to