@@ -64,10 +64,11 @@ Workflow::Workflow(PlugGui::Container * container, BreadCrumbBar * bar, QObject Workflow::~Workflow() { - // delete all the appliances, discarding changes - // saving should have been done at the exit request. here we drop the rest - while (!m_stack.isEmpty()) + // warn if we have any appliance left, shouldn't be + while (!m_stack.isEmpty()) { + qWarning("Workflow::~Workflow: not empty stack. dropping all"); popNode(true); + } // unset the global reference App::workflow = 0; @@ -174,7 +175,9 @@ bool Workflow::requestExit() return true; } - // handle Quit (the distructor will pop apps without saving changes) + // handle Quit without saving + while (!m_stack.isEmpty()) + popNode(true); return true; } App/Workflow.cpp bca1a9f04c9d9280cdf6ba1922e7a74b64eb140c Enrico Ros enrico.ros@gmail.com http://github.com/enricoros/fotowall/commit/063c9b38af5d9651139b1f26a03b503721eb24d7 063c9b38af5d9651139b1f26a03b503721eb24d7 2009-11-06T03:26:30-08:00 2009-11-06T03:26:30-08:00 Small change to enforce good cleanup logic. d1ccc233f8346801ec19776b5e9196bbe6f89439 Enrico Ros enrico.ros@gmail.com