jtamate created this revision.
jtamate added reviewers: Frameworks, dfaure.
Restricted Application added a project: Frameworks.
jtamate requested review of this revision.
This summary will change when the doubts are resolved (if they can be
Implement the first part of a TODO: get rid of the raw KFileItem pointers in
I've added a way to remove the item from the lister in findByUrl, to be
modified and, if needed, added back.
Questions to be answered:
- In findByUrl, the dirItem->rootItem should also be removed?
- fileItem is get twice in slotFileRenamed. Is this patch removing the right
- Should the fileitem removed in slotFileRenamed be added back or is it added
back in emitRefreshItem? If should be added back, to which list?
I know the patched code doesn't follow the original semantics because dolphin
see duplicated entries while renaming takes place.
findByUrl is slow, for example, renaming 50.000 small files, it has to go
through a list of 50.000 items 50.000 times, so renaming that number of files
takes more than an hour.
This patch does not improve findByUrl performance, but allows future work on
To: jtamate, #frameworks, dfaure