https://bugs.kde.org/show_bug.cgi?id=412386

Kai Uwe Broulik <k...@privat.broulik.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-workspace/549374ba5f1e2
                   |                            |b611018c6cfc4a48104a5a8e72c
   Version Fixed In|                            |5.17.1
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED

--- Comment #9 from Kai Uwe Broulik <k...@privat.broulik.de> ---
Git commit 549374ba5f1e2b611018c6cfc4a48104a5a8e72c by Kai Uwe Broulik.
Committed on 22/10/2019 at 08:58.
Pushed by broulik into branch 'Plasma/5.17'.

[Windows Runner] Gather window again when running result

Since we fixed RunnerManager::teardown to actually be emitted, the runner would
throw away its gathered window info and then crash in run().
I couldn't figure out what the proper API was so this is a workaround to keep
it from crashing at least.
However, the runner also didn't check whether the window was actually still
valid by the time run() was called,
so querying for the window again once it is called and checking validity is a
good thing either way imho.
In the future, this probably should be ported to using libtaskmanager :)
FIXED-IN: 5.17.1

Differential Revision: https://phabricator.kde.org/D24849

M  +7    -3    runners/windows/windowsrunner.cpp

https://commits.kde.org/plasma-workspace/549374ba5f1e2b611018c6cfc4a48104a5a8e72c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to