https://bugs.kde.org/show_bug.cgi?id=396339

Max <max...@smoothware.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|DUPLICATE                   |---
     Ever confirmed|0                           |1
                 CC|                            |max...@smoothware.net
             Status|RESOLVED                    |REOPENED

--- Comment #2 from Max <max...@smoothware.net> ---
KMail does saveMainWindowSettings() from mainwindow destructor:
https://cgit.kde.org/kmail.git/tree/src/kmmainwin.cpp#n99

That code should probably be moved to QMainWindow::closeEvent()

void KMMainWin::closeEvent(QCloseEvent *event)
{
    QMainWindow::closeEvent(event);
    if(event->isAccepted()) {
        ... saveMainWindowSettings() ...
    }
}

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to