<div dir="ltr">Thanks for the review.<div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-Man</div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 6:32 PM Stefan Johansson <<a href="mailto:stefan.johansson@oracle.com">stefan.johansson@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"><br>
On 2018-12-12 06:49, Man Cao wrote:<br>
> Thanks for the review!<br>
> <br>
> Is only one review sufficient to push this change?<br>
> In general, is it the case that if any reviewer indicates "trivial", <br>
> only one review is required?<br>
Here's another one :)<br>
<br>
Looks good,<br>
StefanJ<br>
<br>
> <br>
> PS: I read about this page: <br>
> <a href="https://wiki.openjdk.java.net/display/HotSpot/Pushing+a+HotSpot+change" rel="noreferrer" target="_blank">https://wiki.openjdk.java.net/display/HotSpot/Pushing+a+HotSpot+change</a><br>
> but it doesn't say trivial changes only need one review.<br>
> <br>
> -Man<br>
> <br>
> <br>
> On Mon, Dec 10, 2018 at 6:13 PM Thomas Schatzl <br>
> <<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a> <mailto:<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>>> wrote:<br>
> <br>
>     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></div>