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