typelevel / cats

Lightweight, modular, and extensible library for functional programming.
https://typelevel.org/cats/
Other
5.25k stars 1.21k forks source link

[Perf] `Set.contains(value)` vs `alleyCatsSetTraverse.contains_(value)` #4106

Open ronanM opened 2 years ago

ronanM commented 2 years ago
ronanM commented 2 years ago

They may have different behaviors :

Optimization may be impossible ?

armanbilge commented 2 years ago

Ah, good point. Yeah, this probably can't be fixed.

ronanM commented 2 years ago

A warning in the ScalaDoc may be useful for some FoldableOps functions ?