D7438: Also remember which avatars we didn't find
Volker Krause
noreply at phabricator.kde.org
Sun Aug 20 18:42:00 BST 2017
vkrause created this revision.
Restricted Application added a project: KDE PIM.
REVISION SUMMARY
This massively reduces the amount of network requests, at very little
disk and memory cost (1Mb per 20+k of missing avatars with both backends,
60+k for just Gravatar).
It's even more important when having both backends enabled, in that case
we always did a request for the first backend even if the second one could
provide it from the local cache.
REPOSITORY
R88 PIM: Gravatar Library
BRANCH
pending
REVISION DETAIL
https://phabricator.kde.org/D7438
AFFECTED FILES
autotests/gravatarcachetest.cpp
autotests/gravatarcachetest.h
autotests/gravatarresolvurljobtest.cpp
src/CMakeLists.txt
src/job/gravatarresolvurljob.cpp
src/job/gravatarresolvurljob.h
src/misc/gravatarcache.cpp
src/misc/gravatarcache.h
src/misc/hash.cpp
src/misc/hash.h
To: vkrause, #kde_pim
Cc: dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170820/9cdfa272/attachment.html>
More information about the kde-pim
mailing list