dhaumann added inline comments.

INLINE COMMENTS

> repository.cpp:83
>  {
> -    if (candidates.isEmpty())
> -        return Definition();
> -
> -    std::partial_sort(candidates.begin(), candidates.begin() + 1, 
> candidates.end(), [](const Definition &lhs, const Definition &rhs) {
> +    std::partial_sort(definitions.begin(), definitions.begin() + 1, 
> definitions.end(), [](const Definition &lhs, const Definition &rhs) {
>          return lhs.priority() > rhs.priority();

Isn't std::stable_sort what we want here?

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D19200

To: davschul, cullmann, vkrause
Cc: dhaumann, kwrite-devel, kde-frameworks-devel, bmortimer, michaelh, 
genethomas, ngraham, bruns, demsking, cullmann, vkrause, sars

Reply via email to