On Wed, 28 May 2025 17:13:41 GMT, Liam Miller-Cushon <cus...@openjdk.org> wrote:

>> This change overrides mutator methods in the implementation returned by 
>> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
>
> Liam Miller-Cushon has updated the pull request with a new target base due to 
> a merge or a rebase. The pull request now contains 15 commits:
> 
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - Update test/jdk/java/util/Collection/MOAT.java
>    
>    Co-authored-by: Chen Liang <li...@openjdk.org>
>  - Update copyright year
>    
>    and add the bug number to the modified test
>  - Update unmodifiable map javadoc
>  - Also throw UOE for mutators on keySet() and values()
>    
>    and add more test coverage to MOAT.
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - Merge remote-tracking branch 'origin/master' into 
> JDK-8328821-make-clear-consistent
>  - ... and 5 more: https://git.openjdk.org/jdk/compare/b2a61a99...785f728d

For some reason the lines you've added for UOE in MOAT are failing; can you 
check?

I have reviewed the CSR for you. Maybe us, Per Minborg, and Stuart Marks can 
work together to get the two significant collection changes into early 26 and 
broadcast on quality discuss list, as otherwise corpus analysis provides little 
useful information on the compatibility impact.

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

PR Comment: https://git.openjdk.org/jdk/pull/18522#issuecomment-2920978415

Reply via email to