D28340: Figure out nested wayland dismantle

Aleix Pol Gonzalez <[email protected]> Fri, 27 Mar 2020 13:19:31 +0000

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

apol created this revision.
apol added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
apol requested review of this revision.

REVISION SUMMARY
  Fixes a crash we would get, we're changing 2 main things:
  
  - release the event queue first, otherwise we'd serve events to
  
  disappearing objects.
  
  - destroy the connection thread object after the thread has quit.

TEST PLAN
  KWin doesn't explode when closing.

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  plugins/platforms/wayland/wayland_backend.cpp

To: apol, #kwin
Cc: kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart