Review Request 125063: fix for Bug 260869

[Raphael Kugelmann]

> On Sept. 9, 2015, 7:26 nachm., Martin Klapetek wrote:
> >
> 
> Martin Klapetek wrote:
>     Btw. do you have commit access?
> 
> Raphael Kugelmann wrote:
>     I haven't tried it yet.
>     But I think I have no further access rights for the repo.

sorry for this annoying question.
How does this commit work. Is it possible to do it via ChangeBoard or only via git?


- Raphael


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


On Sept. 9, 2015, 8:26 nachm., Raphael Kugelmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125063/
> -----------------------------------------------------------
> 
> (Updated Sept. 9, 2015, 8:26 nachm.)
> 
> 
> 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/7bf37531/attachment.html>