bport added a comment.

  In D28817#647930 <https://phabricator.kde.org/D28817#647930>, @anthonyfieroni 
wrote:
  
  > In D28817#647912 <https://phabricator.kde.org/D28817#647912>, @bport wrote:
  >
  > > I don't think we have a leak, on destructor we delete all view
  > >  qDeleteAll(m_views);
  >
  >
  > At that point, when rootObj is nullptr, view is not added to m_views.
  
  
  Indeed...

REPOSITORY
  R104 KScreen

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

To: bport, #plasma, meven, ervin
Cc: davidedmundson, anthonyfieroni, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, 
ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, ahiemstra, mart

Reply via email to