<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>Some minor comments:</p>
<ul>
<li>General</li>
<ul>
<li>I think you can backtick the UnsupportedOperationException
text so it shows up in monospace format.</li>
</ul>
<li>Specification</li>
<ul>
<li>I think text-wise the wording is fine. IIRC don't we
normally show the proposed changes as a diff? Or is that just
one acceptable format and what's done here is fine too?
Either way the intent is clearly communicated, IMO.</li>
</ul>
</ul>
<p>I'll put myself down as a reviewer.</p>
<p>--Jamil<br>
</p>
<div class="moz-cite-prefix">On 12/11/2020 10:37 AM, Hai-May Chao
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:IjgglQzoV50udvyBf23vfNzLpxdLCPL9papPIK7i3yM=.7427acb9-1aaa-4fd5-8b23-f056a999d86e@github.com">
<pre class="moz-quote-pre" wrap="">On Tue, 8 Dec 2020 18:52:05 GMT, Sean Mullan <a class="moz-txt-link-rfc2396E" href="mailto:mullan@openjdk.org"><mullan@openjdk.org></a> wrote:
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Marked as reviewed by xuelei (Reviewer).
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
This will also require a CSR since you are making some specification changes. I'm not sure if you were trying to get this into 16, but it is probably too late to make JDK 16 since RDP is a couple of days away. You can push to 17/latest once 16 forks and CSR is approved.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
Please review the CSR (JDK-8258114) at:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8258114">https://bugs.openjdk.java.net/browse/JDK-8258114</a>
-------------
PR: <a class="moz-txt-link-freetext" href="https://git.openjdk.java.net/jdk/pull/1701">https://git.openjdk.java.net/jdk/pull/1701</a>
</pre>
</blockquote>
</body>
</html>