On Mon, 23 Jun 2025 07:14:45 GMT, Alan Bateman <al...@openjdk.org> wrote:
> > Note that test j/u/c/tck/ testTimeUnit already includes > > testTimedWait_IllegalMonitorException(). because it was already implied by > > the spec. The change just makes this explicit. Does this require CSR? (If > > so, I'd be happy to do it). > > I don't think it's strictly needed here, Joe Darcy is going to give an > opinion today. Okay -- I've taken a quick look over the proposed changes. While most of the changes are clearly informative (and thus don't need a CSR), I think some of them would fall over the line of what needs a CSR (some changes in TimeUnit.java, RunnableFuture.java, Flow.java). Further work might show those other changes and already implied by other parts of the spec, but I think it is less effort overall just to run a quick CSR for this change. Thanks. ------------- PR Comment: https://git.openjdk.org/jdk/pull/25880#issuecomment-2997849940
