leinir updated this revision to Diff 14080. leinir edited the summary of this revision. leinir added a comment.
Simplify the logic, and only do the xmlloader caching, not the documents themselves which (as apol points out) should be cached already anyway. Not only that, but that codepath was never triggered when i rechecked it after implementing the xmlloader cache, and so, simpler logic is a good thing. Adjusted summary to match. REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5639?vs=13921&id=14080 REVISION DETAIL https://phabricator.kde.org/D5639 AFFECTED FILES src/core/engine.cpp To: leinir, whiting, apol Cc: #frameworks