alex updated this revision to Diff 82946. alex added a comment.
Use enum for caching return type REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29726?vs=82790&id=82946 BRANCH avid_unnecessary_copying (branched from master) REVISION DETAIL https://phabricator.kde.org/D29726 AFFECTED FILES runners/bookmarks/browser.h runners/bookmarks/browsers/chrome.cpp runners/bookmarks/browsers/chromefindprofile.cpp runners/bookmarks/browsers/findprofile.h runners/bookmarks/browsers/firefox.cpp runners/bookmarks/faviconfromblob.cpp runners/bookmarks/fetchsqlite.cpp runners/bookmarks/fetchsqlite.h runners/bookmarks/tests/testchromebookmarks.cpp To: alex, #plasma, ngraham, meven Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart