On Fri, 11 Mar 2022 01:42:19 GMT, XenoAmess <d...@openjdk.java.net> wrote:
>> XenoAmess has updated the pull request incrementally with one additional >> commit since the last revision: >> >> refine test > > oops seems I wronglly added a teat case 0/1 > will delete it later > @XenoAmess I don't think we need `KeyStructure`. It's probably sufficient to > change the map types to `Map<String, String>` and then use > `String.valueOf(i)` to generate strings to use as keys and values. (If we did > need something like `KeyStructure`, it would be a perfect use of record > classes.) Seems good. Will do it at about 12 hours later, when I off work. > Also, I don't think `tableSizeFor` needs quite as many cases. It's sufficient > to test the edge and boundary cases. Well I just don't think there be any cost in adding them... after all they runs fast. ------------- PR: https://git.openjdk.java.net/jdk/pull/7431