NxWM: Fix double deletion of class

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4727 42af7a65-404d-4744-a932-0658087f49c3

ChangeLog.txt

@@ -62,4 +62,7 @@
   some special mouse and keyboard input event handling.
 * NxWM::CTaskbar: Correct the calculation of the physical size of the
   display.
-
+* NxWM::CCalibration: run method must clear m_stop when returning, or you can
+  never restart the Calibration window.
+* NxWM::CTaskbar: On a failure to start an application, the application icon
+  CImage was being deleted twice.

nxwm/src/ccalibration.cxx

@@ -168,8 +168,9 @@ bool CCalibration::run(void)
 
       touchscreenInput(sample);
     }
- 
-   return !m_stop;
+
+  m_stop = false;
+  return true;
 }
 
 /**

nxwm/src/ctaskbar.cxx

@@ -393,8 +393,12 @@ bool CTaskbar::startApplication(IApplication *app, bool minimized)
 
   if (!app->run())
     {
+      // Call stopApplication on a failure to start.  This will call
+      // app->stop() (which is probably not necesary for the application
+      //  but it should be prepared/ to handle it).  stopApplication()
+      // will also removed the icon image from the list and delete it.
+
       stopApplication(app);
-      image->disable();
       return false;
     }