<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<br>
<div class="moz-cite-prefix">On 20/06/2018 01:23, Brian Burkhalter
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:17A3D1A3-9157-444B-8B93-77D3174F9CA8@oracle.com">:<br>
<div>
<blockquote type="cite"><br style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant: normal;
font-weight: normal; letter-spacing: normal; line-height:
normal; orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-stroke-width: 0px;">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant: normal; font-weight:
normal; letter-spacing: normal; line-height: normal;
orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-stroke-width: 0px; float:
none; display: inline !important;">The sizing of the hash
map doesn't take the load factor into account. You might
find `new HashMap<>(1 + (4 * options.length/ 3)` a bit
better with a larger array of options.</span><br
style="font-family: Helvetica; font-size: 12px; font-style:
normal; font-variant: normal; font-weight: normal;
letter-spacing: normal; line-height: normal; orphans: auto;
text-align: start; text-indent: 0px; text-transform: none;
white-space: normal; widows: auto; word-spacing: 0px;
-webkit-text-stroke-width: 0px;">
</blockquote>
<div><br>
</div>
<div>I don’t see what you are referring to. There is no use of
HashMap in these classes. The HashSets are created to be of a
fixed size as they are not resized later. Perhaps Set.of(E…
elements) should be used to create these Sets?</div>
<br>
</div>
<span style="font-family: Helvetica; font-size: 12px; font-style:
normal; font-variant: normal; font-weight: normal;
letter-spacing: normal; line-height: normal; orphans: auto;
text-align: start; text-indent: 0px; text-transform: none;
white-space: normal; widows: auto; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display: inline
!important;"></span><br style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant: normal;
font-weight: normal; letter-spacing: normal; line-height:
normal; orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-stroke-width: 0px;">
</blockquote>
FileSystemProvider L430. This has been changed to create the HashSet
with an initial capacity of len+1 so it's ignoring the load factor.
I'm just saying that the 4 places where this has changed can be
improved to avoid resizing every time. Given that the number of
options is small is almost all cases then it may be simpler to just
use the no-arg constructor for now.<br>
<br>
-Alan<br>
<br>
</body>
</html>