Looks good to me. I don't have anything else to add at this point. I agree the new implementation is much clearer
than the old and it will be great to see it working in the field.

Thanks,

Michael

On 09/05/2019, 14:00, Alan Bateman wrote:
Thanks Chris and Michael for the reviews/help so far. I've uploaded a new webrev to address most of their points here:
   http://cr.openjdk.java.net/~alanb/8221481/2/webrev/index.html

and delta webrev with the changes from 1 to 2 here:
   http://cr.openjdk.java.net/~alanb/8221481/1to2/webrev/index.html

-Alan

Reply via email to