On 16/04/2012 22:18, Kurchi Hazra wrote:
Hi,

Thanks for the reviews. Here is an updated webrev: http://cr.openjdk.java.net/~khazra/7152856/webrev.01
Looks okay to me. I guess "is" and "os" could be locals but it's a minor comment (and no need to generate another webrev if you decide to move them).

-Alan.

Reply via email to