Review Request 109829: Why do we need restoreState?

Aleix Pol Gonzalez aleixpol at gmail.com
Thu Apr 4 23:41:57 UTC 2013


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

(Updated April 4, 2013, 11:41 p.m.)


Review request for KDevelop and Alexander Dymo.


Changes
-------

linked to bug report


Description
-------

I had been hitting some ugly bug recently, where KDevelop would initialize in a weird state where one of the docks was shown but the button wasn't checked, so I had to press the button to hide it (hotkeys didn't work).

I decided to investigate when were the hidden toolviews shown. gdb soon suggested it was in restoreState(). I started to fiddle with the code to see what could be wrong, but I couldn't manage to find out, so I tried to disable it. It worked (surprisingly good I must add, most of the glitches I had when changing areas are gone with this change).

This is not a proper review request, but I was wondering what are we trying to restore there. Toolviews and menues are managed elsewhere, so are the document views. The size isn't being restored there either (load/saveGeometry).


This addresses bug 295536.
    http://bugs.kde.org/show_bug.cgi?id=295536


Diffs
-----

  sublime/mainwindow.cpp 9d55ca6 

Diff: http://git.reviewboard.kde.org/r/109829/diff/


Testing
-------

All tests pass (but testgenerationtest), didn't add any other test.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130404/3fa5b47c/attachment.html>


More information about the KDevelop-devel mailing list