Review Request 125063: fix for Bug 260869

Raphael Kugelmann raphael.ku at gmx.de
Wed Sep 9 19:09:04 UTC 2015


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

Review request for Plasma.


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/255f5dad/attachment.html>


More information about the Plasma-devel mailing list