I think it's ok and less crash-prone to check joinable() rather

than asserting it; I have seen traces which suggest the assert
might be triggering.  In an ideal world we would track down
why that's happening.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/lib/writer.cc b/src/lib/writer.cc
index 17bd21daf..7381d2a90 100644
--- a/src/lib/writer.cc
+++ b/src/lib/writer.cc
@@ -429,8 +429,10 @@ Writer::terminate_thread (bool can_throw)
 	_full_condition.notify_all ();
 	lock.unlock ();
 
-	DCPOMATIC_ASSERT (_thread->joinable ());
- 	_thread->join ();
+	if (_thread->joinable ()) {
+		_thread->join ();
+	}
+
 	if (can_throw) {
 		rethrow ();
 	}