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

Harald Sitter <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/milou/-/commit/cac8ff46a
                   |                            |4f67480ea532399ea5dd26aac7d
                   |                            |9d8f
         Resolution|---                         |FIXED

--- Comment #3 from Harald Sitter <[email protected]> ---
Git commit cac8ff46a4f67480ea532399ea5dd26aac7d9d8f by Harald Sitter.
Committed on 07/10/2025 at 10:22.
Pushed by sitter into branch 'master'.

resultsview: refactor move_category so it doesn't deadlock

- shared impl for both move directions
- have a finite loop condition
- break the loop when finding the first entry of a category

(slightly complicated because the break condition is divergent, I've
opted to handle this inside the move function. another alternative would
have been to pass in an extra conditions function, but that seems just
as complicated and makes for less readable code)

the secret to this function working for both scenarios actually lies in
the caller. the caller suplies a list of indexes and those are either
ascending or descending sorted. effectively allowing us to iterate
always in one direction in the move function.

M  +35   -29   lib/ResultsView.qml

https://invent.kde.org/plasma/milou/-/commit/cac8ff46a4f67480ea532399ea5dd26aac7d9d8f

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

Reply via email to