Get rid of Set<ClassInfo> usages

[mkouba]

Get rid of Set usages

from jandex.

[Ladicek]

I have the same feelings about CompositeIndex containing multiple equivalent but different classes, but I can't tell whether that was an intentional decision or not, and at this point, can't even rule out that there are people depending on it :-)

I mean, I agree it should be investigated, but I'm not 100% convinced yet that they all may safely be eliminated.

[Ladicek]

Those may actually be correct, because in absence of equals/hashCode, a HashMap essentially degenerates into an IdentityHashMap, which may be desired, especially in case of CompositeIndex. That said, I didn't look too deep, reviewing all the places is certainly a good idea.

[mkouba]

may be desired, especially in case of CompositeIndex

Could you be more specific?

TBH I find the fact that a CompositeIndex may contain multiple ClassInfos (same or different) of the same FQCN and the CompositeIndex.getClassByName(DotName) method just returns the first one found very confusing...