cgeo/c-geo-opensource — comment on commit bc5309a0c0797eabaac834d971d27298fa8d786e
File:  line 0
By samueltardieu:

I would be with @blafoo on this one: when you can use a `Set`, it is usually best to do so, especially when you need to remove an element from the set as it is an amortized O(1) operation.

But here, I'm surprised not to see `equals` and `hashCode` overrides for `cgCache`. How would two caches be considered equal and not inserted twice in the set?