davidedmundson requested changes to this revision. davidedmundson added inline comments. This revision now requires changes to proceed.
INLINE COMMENTS > screenpool.cpp:108 > } > Q_ASSERT(m_idForConnector.contains(primary)); > Either this is a valid case to be in, and this assert doesn't make sense. Or we should never be in this case, and the patch doesn't make sense. I don't fully understand the background of how we end up in this situation to say which it is. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D13853 To: hoffmannrobert, #plasma, mart, davidedmundson Cc: davidedmundson, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart