D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.
Alex Nemeth
noreply at phabricator.kde.org
Sun Jan 14 10:24:55 UTC 2018
anemeth marked 8 inline comments as done.
anemeth added a comment.
Restricted Application edited projects, added KWin; removed Plasma.
In https://phabricator.kde.org/D9848#190515, @romangg wrote:
> In https://phabricator.kde.org/D9848#190128, @anemeth wrote:
>
> > Removed .arcconfig and added a whitespace
>
>
> What? Why?
.arcconfig is not needed here and I re-added the whitespace to match the old version.
INLINE COMMENTS
> zzag wrote in blurshader.cpp:86
> Please use `{}`
Most other if statements don't use brackets when there is only a return line and I don't intend to break that tradition, or I might have to correct all of them too.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D9848
To: anemeth, #plasma, #kwin
Cc: romangg, zzag, anthonyfieroni, mart, davidedmundson, fredrik, ngraham, plasma-devel, kwin, #kwin, iodelay, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180114/f0a5adeb/attachment.html>
More information about the Plasma-devel
mailing list