<div>broulik added a comment.</div><br /><div><div><p>Not sure about m_env->corona()->containments()</p>

<p>"this" is const but corona() is not, I see why apol wanted to use foreach all over the place :P but for the non-controversial parts like</p>

<p>const auto foo;</p>

<p>we can really do this...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1588" rel="noreferrer">https://phabricator.kde.org/D1588</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>