<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/109444/">http://git.reviewboard.kde.org/r/109444/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Marco Martin.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">this makes ThemePrivate shared, so Theme instances can be created without relying on it to be a singleton.
as is right now, by default a single "global" theme will be used.
if instead a theme instance is created with a name as a parameter, that instance will be independent.
useGlobalSettings should continue to decide where the configuration will be saved in plasmarc (config group as a property still to be implemented)
The theme writing the config on the global or independent place is something actually independent from the "independent" instance of theme called with new Theme(themeName), so code may be made effectively simpler if that ctor is removed (and with that the themes hash in ThemePrivate): but this would also mean we will not ever try to have two themes in the same process (which may be a good thing, just have to be a decision.)
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/plasma/theme.h <span style="color: grey">(4a28c74)</span></li>
<li>src/plasma/theme.cpp <span style="color: grey">(618d019)</span></li>
<li>src/plasma/private/theme_p.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plasma/CMakeLists.txt <span style="color: grey">(a75d31c)</span></li>
<li>src/plasma/private/theme_p.h <span style="color: grey">(e8b5250)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109444/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>