[KPhotoAlbum] Speeding up annotation
Robert Krawitz
rlk at alum.mit.edu
Sun Jan 5 22:00:53 GMT 2020
On Sun, 05 Jan 2020 21:56:32 +0100, Tobias Leupold wrote:
> Nice :-) It's really cool how you find such performance issues!
Partly it's something I'm interested in, partly I have a very large
database of photos. But there are still multiple problems in this
area (at least 3 more; this was just the worst).
> But maybe, while you're at it, you want to replace ole Q_FOREACH with a C++11
> for ( : ) loop?
Right now the code's a hodgepods, with 160 Q_FOREACH and about 100
(two different regexps yielded 102 and 103 respectively) for ( : )
loops. I agree that we want to fix that; the question is do we want
to do it piecemeal or in one shot.
> Am Sonntag, 5. Januar 2020, 20:23:50 CET schrieb Robert Krawitz:
>> I found one easy fix in the selection code after the annotations are
>> changed. It's still very slow and may be worse than linear, but at
>> least one real problem is cleaned up.
>>
>> Given where we are in the release cycle, I want more review before
>> pushing it.
--
Robert Krawitz <rlk at alum.mit.edu>
*** MIT Engineers A Proud Tradition http://mitathletics.com ***
Member of the League for Programming Freedom -- http://ProgFree.org
Project lead for Gutenprint -- http://gimp-print.sourceforge.net
"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton
More information about the Kphotoalbum
mailing list