Some minor comments:

 * General
     o I think you can backtick the UnsupportedOperationException text
       so it shows up in monospace format.
 * Specification
     o 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.

I'll put myself down as a reviewer.

--Jamil

On 12/11/2020 10:37 AM, Hai-May Chao wrote:
On Tue, 8 Dec 2020 18:52:05 GMT, Sean Mullan <mul...@openjdk.org> wrote:

Marked as reviewed by xuelei (Reviewer).
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.
Please review the CSR (JDK-8258114) at:
https://bugs.openjdk.java.net/browse/JDK-8258114

-------------

PR: https://git.openjdk.java.net/jdk/pull/1701

Reply via email to