<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124212/">https://git.reviewboard.kde.org/r/124212/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 30th, 2015, 4:59 p.m. CEST, <b>Aleix Pol Gonzalez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">In general, I like it.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Maybe now that we're putting the files in separate directories, we can drop the _mac postfixes?</p></pre>
</blockquote>
<p>On June 30th, 2015, 5:08 p.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I like the idea, but that won't be possible for all cases. E.g. with kkeyserver_foo it's still needed as they also get installed. Also with the private headers it might be tricky (I have another patch on top of that one to change to a plugin architecture and e.g. kwindowsystem_p_x11.h needs to include kwindowsystem_p.h). I'll have a look on it in my on top of this patch for the plugins what can be done.</p></pre>
</blockquote>
<p>On June 30th, 2015, 5:11 p.m. CEST, <b>Aleix Pol Gonzalez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Right. Still, it would be ideal to rename the files only once, so it's easier to read the file's history.</p></pre>
</blockquote>
<p>On July 1st, 2015, 8:19 a.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Ah that was a misunderstanding. Of course in the first commit. I just wanted to check in the commit on top of it how much it makes sense.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I have it prepared now. At least the .cpp files could be renamed. For the .h files I don't see a solution, if you have ideas please let me know.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Maybe we can always include the plugin files using the platform? such as <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">#include "mac/fu.h</code></p></pre>
<br />
<p>- Aleix</p>
<br />
<p>On July 1st, 2015, 8:20 a.m. CEST, Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDE Frameworks.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated July 1, 2015, 8:20 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kwindowsystem
</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;">In the src directory we introduce a platforms directory with a specific
* osx
* wayland
* windows
* xcb
sub directory. The platform specific source and header files are moved
into those directories.
This is a preparation step to move the platform specific behavior into
plugins which get loaded at runtime. Such a change is required to
support more platforms in future for which kwindowsystem cannot provide
support directly. An example is proper Wayland support on the Plasma
platform. It needs to provide its own implementationn which has to
differ from the generic Wayland implementation.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">compiles, installs and autotest pass on platform Linux/xcb</p></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>autotests/CMakeLists.txt <span style="color: grey">(dda0af259b292ee5e526bb9166811fc1b376f388)</span></li>
<li>src/CMakeLists.txt <span style="color: grey">(ff2ce392ecd7969eb94543528c7a670ea0fcd870)</span></li>
<li>src/fixx11h.h <span style="color: grey">()</span></li>
<li>src/kkeyserver_mac.h <span style="color: grey">()</span></li>
<li>src/kkeyserver_mac.cpp <span style="color: grey">()</span></li>
<li>src/kkeyserver_win.h <span style="color: grey">()</span></li>
<li>src/kkeyserver_win.cpp <span style="color: grey">()</span></li>
<li>src/kkeyserver_x11.h <span style="color: grey">()</span></li>
<li>src/kkeyserver_x11.cpp <span style="color: grey">()</span></li>
<li>src/kmanagerselection.h <span style="color: grey">()</span></li>
<li>src/kselectionowner.h <span style="color: grey">()</span></li>
<li>src/kselectionowner.cpp <span style="color: grey">()</span></li>
<li>src/kselectionwatcher.h <span style="color: grey">()</span></li>
<li>src/kselectionwatcher.cpp <span style="color: grey">()</span></li>
<li>src/kwindoweffects_x11.cpp <span style="color: grey">()</span></li>
<li>src/kwindowinfo_mac.cpp <span style="color: grey">()</span></li>
<li>src/kwindowinfo_mac_p.h <span style="color: grey">()</span></li>
<li>src/kwindowinfo_p_x11.h <span style="color: grey">()</span></li>
<li>src/kwindowinfo_win.cpp <span style="color: grey">()</span></li>
<li>src/kwindowinfo_x11.cpp <span style="color: grey">()</span></li>
<li>src/kwindowsystem.cpp <span style="color: grey">(789132e1b4883dd54218d29af9710dedfe6218e1)</span></li>
<li>src/kwindowsystem_mac.cpp <span style="color: grey">()</span></li>
<li>src/kwindowsystem_p_wayland.h <span style="color: grey">()</span></li>
<li>src/kwindowsystem_p_x11.h <span style="color: grey">()</span></li>
<li>src/kwindowsystem_wayland.cpp <span style="color: grey">()</span></li>
<li>src/kwindowsystem_win.cpp <span style="color: grey">()</span></li>
<li>src/kwindowsystem_x11.cpp <span style="color: grey">()</span></li>
<li>src/kxerrorhandler.cpp <span style="color: grey">()</span></li>
<li>src/kxerrorhandler_p.h <span style="color: grey">()</span></li>
<li>src/kxmessages.h <span style="color: grey">()</span></li>
<li>src/kxmessages.cpp <span style="color: grey">()</span></li>
<li>src/kxutils.cpp <span style="color: grey">()</span></li>
<li>src/kxutils_p.h <span style="color: grey">()</span></li>
<li>src/netwm.h <span style="color: grey">()</span></li>
<li>src/netwm.cpp <span style="color: grey">()</span></li>
<li>src/netwm_p.h <span style="color: grey">()</span></li>
<li>src/platforms/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/platforms/osx/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/platforms/wayland/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/platforms/windows/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/platforms/xcb/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124212/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>