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. Thanks for expanding the test coverage for this.

-Alan.

Reply via email to