On 18/02/2023 05:43, Han Li wrote:
On Feb 17, 2023, at 17:17, Rémy Maucherat <r...@apache.org> wrote:
On Fri, Feb 17, 2023 at 4:32 AM <li...@apache.org> wrote:
- ciphers.removeAll(movedCiphers);
+ movedCiphers.forEach(ciphers::remove);
Ok for some of them maybe, but I don't understand why one this is better.
See:https://www.baeldung.com/java-hashset-removeall-performance
<https://www.baeldung.com/java-hashset-removeall-performance>
It’s just possible to avoid some potential performance issues. So i replaced
old writing style with this. :)
Those performance issues are when the collection the items are being
removed from is an ArrayList. These all look to be LinkedHashSet.
There may be a very small gain from these changes as it skips some logic
in removeAll() that isn't necessary in these instances.
Mark
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org