Also remember which avatars we didn't find

Authored by vkrause on Aug 20 2017, 5:31 PM.

Description

Also remember which avatars we didn't find

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.

Reviewers: KDE PIM, mlaurent

Reviewed By: KDE PIM, mlaurent

Subscribers: mlaurent

Tags: KDE PIM

Differential Revision: https://phabricator.kde.org/D7438

Details

Committed
vkrauseAug 21 2017, 4:29 PM
Reviewer
KDE PIM
Differential Revision
D7438: Also remember which avatars we didn't find
Parents
R88:a43207a9a555: Add unit test for cache lookup
Branches
Unknown
Tags
Unknown