markg created this revision.
markg added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
markg requested review of this revision.
Small performance improvement of just not caching the targetUrlList() result.
The only use for that list was returning it in
KFileItemListProperties::urlList() so we might as well just call
m_items.targetUrlList() there and not save it locally.
This came up as redundant list creation when figuring out where all the QUrl
objects came from in a drag/drop action in Dolphin.
I did not benchmark this, seems rather trivial.
Ran the testcases (most relevant is probablt KFileItemTest), all pass just
KFileItemListProperties_optimizations (branched from master)
To: markg, dfaure
Cc: #frameworks, michaelh, ngraham