ComparatorCompat about lightweight-stream-api

[aNNiMON]

Hi,

Thanks for raising such an important issue.

1. In Java 8 Comparators class marked as package private, so I preferred not to use it and chose ComparatorCompat as well as RandomCompat. I understand that Comparators looks much clearer and it is shorter by 5 characters... My bad.

2. There is chain method:

ComparatorCompat
        .chain(comparator)
        .reversed()
        .thenComparing(...)
        .comparator();

See more examples.

Your approach with an abstract class is much better, I'll think about using it. The only problem is that Comparators implements Comparator is non-obvious. Comparators sounds like an utility class or container, not Comparator instance itself. And I have no idea how to call this class shorter and clearer.

[BattleShipPark]

Hi,
I missed Java 8 Comparators, and agree ComparatorCompat is a good alternative name.

Then, what about ComparatorCompat implements Comparator? I am not sure such an inheritance makes no side effect ever but I think its approach is worth considering.

Thanks.

[aNNiMON]

Okay, I managed to make changes without loosing source compatibility.

@BattleShipPark, thank you for that suggestion.