[lmi-commits] [5841] Call base class's OnExit() in override

[Greg Chicares]

Revision: 5841
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5841
Author:   chicares
Date:     2013-11-20 17:59:20 +0000 (Wed, 20 Nov 2013)
Log Message:
-----------
Call base class's OnExit() in override

Modified Paths:
--------------
    lmi/trunk/ChangeLog
    lmi/trunk/main_wx.cpp

Modified: lmi/trunk/ChangeLog
===================================================================
--- lmi/trunk/ChangeLog 2013-11-20 17:51:44 UTC (rev 5840)
+++ lmi/trunk/ChangeLog 2013-11-20 17:59:20 UTC (rev 5841)
@@ -32961,3 +32961,14 @@
   version.hpp
 Designate release candidate 5838.
 
+20131120T1751Z <address@hidden> [542]
+
+  main_wx.cpp
+Initialize all data members in ctor-initializer for safety.
+
+20131120T1759Z <address@hidden> [542]
+
+  main_wx.cpp
+Call base class's OnExit() in override. See:
+  http://lists.nongnu.org/archive/html/lmi/2013-11/msg00020.html
+

Modified: lmi/trunk/main_wx.cpp
===================================================================
--- lmi/trunk/main_wx.cpp       2013-11-20 17:51:44 UTC (rev 5840)
+++ lmi/trunk/main_wx.cpp       2013-11-20 17:59:20 UTC (rev 5841)
@@ -608,12 +608,16 @@
     return true;
 }
 
+/// wxApp::OnExit() override.
+///
+/// Call the base class's implementation--see:
+///   http://lists.nongnu.org/archive/html/lmi/2013-11/msg00020.html
+
 int Skeleton::OnExit()
 {
     doc_manager_->FileHistorySave(*config_);
     delete doc_manager_;
-    delete config_;
-    return 0;
+    return wxApp::OnExit(); // Deletes config_.
 }
 
 // WX !! An exception thrown anywhere in this function, even right