D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

[Vlad Zagorodniy]

zzag added a comment.
Restricted Application edited projects, added KWin; removed Plasma.


  In https://phabricator.kde.org/D9848#190964, @luebking wrote:
  
  > I'm not actually smiling ...
  >
  > Anyway, instead of using three vectors it's strongly suggest to use one vector (or rather array) of a struct.
  >  a) you can access the inner values by names
  >  b) it's far more elegant to define = { {1, 2, 3}, {4, 5, 6}, ...}
  
  
  c) it's way way elegant to calculate required params(like PixiJS does for determining kawase kernels https://github.com/pixijs/pixi-filters/blob/master/filters/kawase-blur/src/KawaseBlurFilter.js#L84-L100)

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D9848

To: anemeth, #plasma, #kwin
Cc: luebking, broulik, 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/22018a5a/attachment-0001.html>