Review Request 125063: fix for Bug 260869

[Raphael Kugelmann]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125063/
-----------------------------------------------------------

(Updated Sept. 9, 2015, 10:17 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit cee5e903812b5d2fae9e5be52440b8056626e95f by Martin Klapetek on behalf of Raphael Kugelmann to branch Plasma/5.4.


Repository: ksysguard


Description
-------

Bugfix for Bug 260869 - [steps] System Monitor crashes while changing sensors


Diffs
-----

  gui/SensorDisplayLib/FancyPlotter.cpp 6c2e8d14e0217c324845dc1b9b34481f3c6be34c 

Diff: https://git.reviewboard.kde.org/r/125063/diff/


Testing
-------

Fail test procedure:
1. create a FancyPlotter View with an Sensor
2. open the settings for this FancyPlotter view and goto Sensor Tab
3. add in the MainWindow (in the background) a new Sensor
4. Close the Setting window by pressing the OK button
--> Crash

The Bugfix makes the mainwindow not editable while the settings dialog is open.
Therefore the written test above cannot be performed anymore.
The Fail case cannot be reached anymore.


Thanks,

Raphael Kugelmann

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150909/ce2d87bb/attachment.html>