From 2c62f1ebb8d8b5be7db966db9dc3caa02e4d8edb Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Wed, 27 Nov 2024 23:32:17 +0100
Subject: Cleanup: inline shutdown() in the only place it's called.

---
 src/lib/grok/context.h | 38 ++++++++++++++++----------------------
 1 file changed, 16 insertions(+), 22 deletions(-)

diff --git a/src/lib/grok/context.h b/src/lib/grok/context.h
index 77e11ec07..9b249e8d8 100644
--- a/src/lib/grok/context.h
+++ b/src/lib/grok/context.h
@@ -177,7 +177,22 @@ public:
 
 	~GrokContext()
 	{
-		shutdown();
+		if (!_messenger) {
+			return;
+		}
+
+		std::unique_lock<std::mutex> lk_global(launchMutex);
+
+		if (!_messenger) {
+			return;
+		}
+
+		if (_launched) {
+			_messenger->shutdown();
+		}
+
+		delete _messenger;
+		_messenger = nullptr;
 	}
 
 	bool launch(DCPVideo dcpv, int device)
@@ -256,27 +271,6 @@ public:
 	}
 
 private:
-
-	void shutdown()
-	{
-		if (!_messenger) {
-			return;
-		}
-
-		std::unique_lock<std::mutex> lk_global(launchMutex);
-
-		if (!_messenger) {
-			return;
-		}
-
-		if (_launched) {
-			_messenger->shutdown();
-		}
-
-		delete _messenger;
-		_messenger = nullptr;
-	}
-
 	void frame_done()
 	{
 		_dcpomatic_context->history.event();
-- 
cgit v1.2.3