Change itemsInUse, itemsCached and DirectoryDataHash keys from QString to QUrl, avoiding as many QUrl toString() as possible.
Details
Details
- Reviewers
dfaure - Group Reviewers
Frameworks - Commits
- R241:ae08cc2c983c: [kcoredirlister] Remove as many url.toString() as possible
It passes the autotests using 20 msecs less.
Diff Detail
Diff Detail
- Repository
- R241 KIO
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
Yep, this makes sense these days (back when this was written, QUrl had no hashing function, and my first attempt at writing one was to call qHash(url.toString()) -- how horribly slow!!)