D9128: [knewstuff] Respect global variable

Michael Pyne noreply at phabricator.kde.org
Wed Dec 6 23:47:44 UTC 2017


mpyne accepted this revision.
mpyne added a comment.
This revision is now accepted and ready to land.


  I think the change looks good.  But if the cache here is holding a *weak* pointer to a `Cache` then every user of this s_cache should be checking for null pointers each time, right?  We can do the improved lifetime checking here but I think it would also have worked fine to check weak pointers before using them.  Though maybe weak pointers were just there to avoid circular references?

REPOSITORY
  R304 KNewStuff

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

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


More information about the Kde-frameworks-devel mailing list