* Added "bool wait" parameter to ModelLoader::Abort(), allowing to optionally

wait for the loader thread.
* MainWindow::Quit() aborts the loader now.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@30425 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/apps/debuganalyzer/gui/main_window/MainWindow.cpp

@@ -106,6 +106,9 @@ printf("MSG_MODEL_LOADED_FAILED/MSG_MODEL_LOADED_ABORTED\n");
 void
 MainWindow::Quit()
 {
+	if (fModelLoader != NULL)
+		fModelLoader->Abort(true);
+
 	fSubWindowManager->Broadcast(B_QUIT_REQUESTED);
 	be_app->PostMessage(MSG_WINDOW_QUIT);
 

src/apps/debuganalyzer/model_loader/ModelLoader.cpp

@@ -84,10 +84,7 @@ ModelLoader::ModelLoader(DataSource* dataSource,
 
 ModelLoader::~ModelLoader()
 {
-	if (fLoaderThread >= 0) {
+	Abort(true);
-		Abort();
-		wait_for_thread(fLoaderThread, NULL);
-	}
 
 	delete fDataSource;
 	delete fModel;
@@ -128,14 +125,22 @@ ModelLoader::StartLoading()
 
 
 void
-ModelLoader::Abort()
+ModelLoader::Abort(bool wait)
 {
 	AutoLocker<BLocker> locker(fLock);
 
-	if (!fLoading || fAborted)
+	if (fLoaderThread < 0)
 		return;
 
-	fAborted = true;
+	thread_id thread = fLoaderThread;
+
+	if (fLoading)
+		fAborted = true;
+
+	locker.Unlock();
+
+	if (wait)
+		wait_for_thread(thread, NULL);
 }
 
 

src/apps/debuganalyzer/model_loader/ModelLoader.h

@@ -27,7 +27,7 @@ public:
 								~ModelLoader();
 
 			status_t			StartLoading();
-			void				Abort();
+			void				Abort(bool wait);
 
 			Model*				DetachModel();