Differential D5616

Set up a cache for attica
ClosedPublic
Actions

Authored by apol on Apr 27 2017, 10:54 AM.

Details

Reviewers
leinir
Group Reviewers
Plasma
Commits
R119:3189f81aa6f3: Set up a cache for attica
Summary

Reuses the same code we used to cache resources within Discover

BUG: 379193

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
apol created this revision.Apr 27 2017, 10:54 AM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 27 2017, 10:54 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
jgrulich added a subscriber: jgrulich.Apr 28 2017, 11:14 AM

Looks good to me.

+1

apol edited the summary of this revision. (Show Details)Apr 28 2017, 11:25 AM
leinir accepted this revision.Apr 28 2017, 12:56 PM

Looks good to me! :) i am employing a similar method in an upcoming KNS patch, so i will equalise the paths and whatnot in that one, so things are nice and clear :)

This revision is now accepted and ready to land.Apr 28 2017, 12:56 PM
Closed by commit R119:3189f81aa6f3: Set up a cache for attica (authored by apol). · Explain WhyApr 28 2017, 12:59 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 13918

View Options

attica-kde/kdeplugin/kdeplatformdependent.cpp

Loading...