<table><tr><td style="">graesslin created this revision.<br />graesslin added reviewers: KWin, Plasma.<br />Restricted Application added a project: KWin.<br />Restricted Application added subscribers: kwin, plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3402" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The feature has always been considered experimental. Unfortunately it is<br />
completely unmaintained and hasn't seen any commits in years. It<br />
requires kolor-manager to function, but that has not seen a release<br />
based on frameworks yet. This makes it difficult to maintain. In fact I<br />
have never been able from the introduction till now to setup a color<br />
corrected system. One needs kolor-manager and oyranos and especially the<br />
latter is hardly available on any linux distribution (e.g. not on the<br />
Debian/Ubuntu systems).</p>

<p>Due to being unmaintained color correction in KWin did not keep up with<br />
recent changes. Neither did it see any updates during the xlib->xcb<br />
port, nor during the Wayland port. Especially the Wayland port with the<br />
rendering changes make it unlikely to function correctly. E.g. Wayland<br />
introduced a proper per-screen rendering, while color correction did a<br />
"fake" per screen rendering. How that is going to work in combination is<br />
something nobody ever tried. Now after the introduction of proper<br />
per-screen rendering the solution would be to port color correction to<br />
the new api, but that never happened.</p>

<p>Color correction also modified the shaders, but a newer shader API got<br />
introduced some time ago. Whether the color correction shader support<br />
that or not, is unknown to me. Also which shader language versions are<br />
supported. I know it was based on 3d texture support, which back on<br />
introduction was partially lacking in OpenGL ES. Nowadays that changed,<br />
but color correction didn't update.</p>

<p>Last but not least it is completely X11 based and there is no work on<br />
how to make it work with Wayland.</p>

<p>Given all the problems, especially the fact that it is unmaintained and<br />
cannot be setup on my system, means to me that the only solution is to<br />
remove it.</p>

<p>I'm open to having it reintroduced in future, but only if the<br />
availability on Linux distributions gets addressed before. As long as<br />
major linux distributions do not ship this feature, it should not be in<br />
KWin. Given that I must say that it was a mistake to add it in the first<br />
place and I need to point out that I was against the merge back then.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>drop-color-correction</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3402" rel="noreferrer">https://phabricator.kde.org/D3402</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcmkwin/kwincompositing/compositing.cpp<br />
kcmkwin/kwincompositing/compositing.h<br />
kcmkwin/kwincompositing/compositing.ui<br />
kcmkwin/kwincompositing/main.cpp<br />
kwin.kcfg<br />
libkwineffects/CMakeLists.txt<br />
libkwineffects/kwinglcolorcorrection.cpp<br />
libkwineffects/kwinglcolorcorrection.h<br />
libkwineffects/kwinglcolorcorrection_p.h<br />
libkwineffects/kwinglutils.cpp<br />
libkwineffects/kwinglutils.h<br />
options.cpp<br />
options.h<br />
scene_opengl.cpp<br />
scene_opengl.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graesslin, KWin, Plasma<br /><strong>Cc: </strong>plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas<br /></div>