D17069: [x11] Emit clientRemoved after client was removed

Vlad Zagorodniy <[email protected]> Thu, 22 Nov 2018 10:11:20 +0000

Newsgroups	gmane.comp.kde.devel.kwin
Message-ID	<[email protected]>

zzag added a comment.


  In D17069#363989 <https://phabricator.kde.org/D17069#363989>, @graesslin wrote:
  
  > I'm wondering what the intention was to have it that early?
  
  
  Me too. That's quite unusual place to emit the signal (at least, a comment should have been added why).
  
  I doubt there was some reasoning behind emitting clientRemoved that early. To me, it looks like it was added in a hurry.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: graesslin, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart