Review Request 123654: Improve performance of KFileItem and KDirSortFilterProxyModel
David Faure
faure at kde.org
Sun May 10 22:43:19 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123654/#review80189
-----------------------------------------------------------
Yes, please split this out into two commits. I can approve the KFileItem change, I don't know about the rest.
src/core/kfileitem.cpp (line 1115)
<https://git.reviewboard.kde.org/r/123654/#comment55021>
I had the same thought as Milian... but I wonder why I wrote that comment. m_strText is the displayName, m_strName is not. Maybe I confused the two when I wrote it. That's my best guess right now.
Mark: the first commit is about the current directory ("."), the second commit is about urls without a filename, how does it make the first one obsolete? I don't see the relation.
- David Faure
On May 6, 2015, 2:35 a.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123654/
> -----------------------------------------------------------
>
> (Updated May 6, 2015, 2:35 a.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: kio
>
>
> Description
> -------
>
> Reduces some operations in KDirSortFilterProxyModel.
> Removes usage of QUrl::fileName() in KFileItem::isHidden(), it comprised 75% of the time spent running the benchmark in here.
>
>
> Diffs
> -----
>
> autotests/CMakeLists.txt 010074d
> autotests/kdirsortfilterproxymodel_benchmark.cpp PRE-CREATION
> src/core/kfileitem.cpp 344ac67
> src/filewidgets/kdirsortfilterproxymodel.cpp 22ac025
>
> Diff: https://git.reviewboard.kde.org/r/123654/diff/
>
>
> Testing
> -------
>
> Everything seems to be working still, including tests.
>
> There was a comment about trash:/ triggering an assert, but I couldn't reproduce.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150510/39696aa2/attachment.html>
More information about the Kde-frameworks-devel
mailing list