<div dir="ltr"><div dir="ltr">Thanks for the review!<div><br><div>Is only one review sufficient to push this change?</div><div>In general, is it the case that if any reviewer indicates "trivial", only one review is required?</div><div><br><div><div>PS: I read about this page: <a href="https://wiki.openjdk.java.net/display/HotSpot/Pushing+a+HotSpot+change">https://wiki.openjdk.java.net/display/HotSpot/Pushing+a+HotSpot+change</a></div><div>but it doesn't say trivial changes only need one review.</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">-Man</div></div></div><br></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 10, 2018 at 6:13 PM Thomas Schatzl <<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On Mon, 2018-12-10 at 18:04 +0800, Man Cao wrote:<br>
> Hi all,<br>
> <br>
> Can I have reviews for this small cleanup?<br>
> <br>
> Webrev: <a href="https://cr.openjdk.java.net/~manc/8215114/webrev.00/" rel="noreferrer" target="_blank">https://cr.openjdk.java.net/~manc/8215114/webrev.00/</a><br>
> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8215114" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8215114</a><br>
> <br>
<br>
  looks good and trivial.<br>
<br>
Thomas<br>
<br>
<br>
</blockquote></div>