[krusader] [Bug 527888] Descending sort uses a comparator that is not a strict weak ordering (undefined behaviour in std::stable_sort)

[Attila Torda]

https://bugs.kde.org/show_bug.cgi?id=527888

--- Comment #2 from Attila Torda <attila.to...@proton.me> ---
Just to clarify, I understand that there is normally only one .. entry.
However, the comparator can still be called with the same dummy entry as both
arguments (itemLessThan(dummy, dummy)).

In ascending mode, the current implementation returns true in this case,
violating the strict weak ordering requirement (specifically, irreflexivity).

So I agree that the additional check is necessary:

-- 
You are receiving this mail because:
You are watching all bug changes.