D9318: [knewstuff] Limit request cache size

[Anthony Fieroni]

anthonyfieroni added a comment.


  In https://phabricator.kde.org/D9318#179477, @dfaure wrote:
  
  > Technically no, since it doesn't change the size of the class itself (and requestCache is private and isn't used in any inline method).
  
  
  You are right, size will not change.
  
  In https://phabricator.kde.org/D9318#179477, @dfaure wrote:
  
  > The benefit of QSet over a linear search only starts after a large enough amount of elements in the set though. I have no idea about typical sizes here.
  
  
  It can be really huge depend of requests intensity.

REPOSITORY
  R304 KNewStuff

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

To: anthonyfieroni, leinir, #frameworks
Cc: dfaure, broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171214/25398828/attachment.html>