Yes! Looks good to me Patrick.

Just remove 8246132 from the @bug list.
You don't list the bug number when the bug is in the test itself.
That's why the bug has a noreg-self keyword.
No need for a new webrev.

best regards,

-- daniel


On 05/06/2020 12:36, Patrick Concannon wrote:
Hi Alan,

That’s much more concise. Thanks for the tip!

I’ve incorporated it into the updated webrev below.

http://cr.openjdk.java.net/~pconcannon/8246132/webrevs/webrev.01/

Kind regards,
Patrick


Reply via email to