(svn r15006) -Codechange: throw a real instance of a class, instead of '0' (which can also be a throw from within a thread for what ever reason)

src/thread.h

@@ -7,6 +7,8 @@
 
 typedef void (*OTTDThreadFunc)(void *);
 
+class OTTDThreadExitSignal { };
+
 /**
  * A Thread Object which works on all our supported OSes.
  */

src/thread_morphos.cpp

@@ -153,7 +153,7 @@ public:
 
 		if (NewGetTaskAttrs(NULL, &msg, sizeof(struct OTTDThreadStartupMessage *), TASKINFOTYPE_STARTUPMSG, TAG_DONE) && msg != NULL) {
 			/* For now we terminate by throwing an error, gives much cleaner cleanup */
-			throw 0;
+			throw OTTDThreadExitSignal();
 		}
 
 		return true;
@@ -203,8 +203,10 @@ private:
 		if (NewGetTaskAttrs(NULL, &msg, sizeof(struct OTTDThreadStartupMessage *), TASKINFOTYPE_STARTUPMSG, TAG_DONE) && msg != NULL) {
 			try {
 				msg->func(msg->arg);
-			} catch(...) {
+			} catch(OTTDThreadExitSignal e) {
 				KPutStr("[Child] Returned to main()\n");
+			} catch(...) {
+				NOT_REACHED();
 			}
 		}
 

src/thread_pthread.cpp

@@ -92,7 +92,7 @@ public:
 		if (!IsRunning()) return false;
 
 		/* For now we terminate by throwing an error, gives much cleaner cleanup */
-		throw 0;
+		throw OTTDThreadExitSignal();
 	}
 
 	/* virtual */ void Join()
@@ -137,7 +137,9 @@ private:
 		/* Call the proc of the creator to continue this thread */
 		try {
 			m_proc(m_param);
+		} catch (OTTDThreadExitSignal e) {
 		} catch (...) {
+			NOT_REACHED();
 		}
 
 		/* Notify threads waiting for our completion */

src/thread_win32.cpp

@@ -87,7 +87,7 @@ public:
 		if (!IsRunning()) return false;
 
 		/* For now we terminate by throwing an error, gives much cleaner cleanup */
-		throw 0;
+		throw OTTDThreadExitSignal();
 	}
 
 	/* virtual */ void Join()
@@ -128,7 +128,9 @@ private:
 	{
 		try {
 			m_proc(m_param);
+		} catch (OTTDThreadExitSignal) {
 		} catch (...) {
+			NOT_REACHED();
 		}
 	}
 };