D23918: RFC: Drop Outputs class

Vlad Zahorodnii noreply at phabricator.kde.org
Fri Sep 13 12:04:58 BST 2019


zzag marked an inline comment as done.
zzag added a comment.


  In D23918#530493 <https://phabricator.kde.org/D23918#530493>, @davidedmundson wrote:
  
  > Doesn't solve anything and now we have casts all over the place.
  
  
  Well, it doesn't cure cancer, that's for sure, however it fixes a code smell.
  
  Can you provide some arguments why we **must** inherit from QVector<T>? C++ templates are invariant so we have to cast manually.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: romangg, davidedmundson, alexeymin, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190913/734d6dfe/attachment.html>


More information about the kwin mailing list