Review Request: Use QDirIterator in KStandardDirs

Jarosław Staniek staniek-RoXCvvDuEio at public.gmane.org
Thu Jun 9 19:49:14 BST 2011



> On June 9, 2011, 9:49 a.m., Aaron J. Seigo wrote:
> > my biggest question with this change would be performance. QFileInfo is not fast, and to make matters "worse" this code in KStandardDirs influences start up speed. have you measured the performance after this change relative to before?
> > 
> > with Qt5 this may become a moot issue with the changes coming in the QFile backends ... but i don't think it makes sense to introduce possible performance regressions on code that currently works.
> > 
> > is there a platform for which the current code fails?
> 
> Bernhard Beschow wrote:
>     I did the patch solely for simplicity and portability reasons, and perhaps (or hopefully) also in the Platform 11 spirit. :) After all, "outsourcing" platform-specific details to Qt wherever we can seems like a good idea to me, such that we can take advantage of new platforms much quicker.
>     
>     That said, I couldn't measure any performance difference when logging into a plasma session. In both cases, it takes ~13 secs for the wallpaper to appear, and ~30 secs until the KWallet password dialog appears.

I understand the reasoning but the fact that you did not properly measure the differences shows such pieces of code would be better left untouched untill there is convincing data. KStandardDirs is critical place for KDE apps. IMHO simplifing such lower-level internal (tested!) code may not pay off.

Thanks for your effort.


- Jarosław


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101554/#review3799
-----------------------------------------------------------


On June 9, 2011, 7:06 a.m., Bernhard Beschow wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101554/
> -----------------------------------------------------------
> 
> (Updated June 9, 2011, 7:06 a.m.)
> 
> 
> Review request for kdelibs, kdewin and David Faure.
> 
> 
> Summary
> -------
> 
> Increase portability of kdecore a tiny bit by making KStandardDirs use QDirIterator rater than platform-specific code.
> 
> 
> Diffs
> -----
> 
>   kdecore/kernel/kstandarddirs.cpp ba90270 
> 
> Diff: http://git.reviewboard.kde.org/r/101554/diff
> 
> 
> Testing
> -------
> 
> I'm running my platform 4.6 with this patch and haven't noticed any regressions yet.
> 
> 
> Thanks,
> 
> Bernhard
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110609/44c66a9f/attachment.htm>
-------------- next part --------------
_______________________________________________
Kde-windows mailing list
Kde-windows-RoXCvvDuEio at public.gmane.org
https://mail.kde.org/mailman/listinfo/kde-windows


More information about the kde-core-devel mailing list