Review Request 125063: fix for Bug 260869

[Martin Klapetek]

> On Sept. 9, 2015, 9:26 p.m., 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.
> 
> Raphael Kugelmann wrote:
>     sorry for this annoying question.
>     How does this commit work. Is it possible to do it via ChangeBoard or only via git?
> 
> Raphael Kugelmann wrote:
>     a "git push" results into access denied, so I think I haven't commit access.
>     Can you please do the commit? Thanks!

Will do. Usually if you do couple good patches, you can request a developer account, so keep 'em coming ;)


- Martin


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


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