D13135: Introduce Storage::articlesForCache and more lazy-loading

Laurent Montel noreply at phabricator.kde.org
Wed Jun 13 07:21:42 BST 2018


mlaurent added inline comments.

INLINE COMMENTS

> mlaurent wrote in feedstorage.h:45
> const QDateTime &

still not done

> feedstoragemk4impl.cpp:207
> +    QVector<SmallArticle> result;
> +    int size = d->archiveView.GetSize();
> +    result.reserve(size);

const int

> mlaurent wrote in article.cpp:266
> why not assign d directly here ?

why not assign d directly here ?

> mlaurent wrote in feedstoragedummyimpl.cpp:125
> const QString & ?

still not done

> mlaurent wrote in feed.cpp:264
> const ... & ?

still not done

REPOSITORY
  R201 Akregator

REVISION DETAIL
  https://phabricator.kde.org/D13135

To: pinaraf, cgiboudeaux
Cc: mlaurent, kde-pim, dvasin, rodsevich, winterz, vkrause, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20180613/8a3608de/attachment.html>


More information about the kde-pim mailing list