From cc15132ccc17de0e97006713692a23419e58f369 Mon Sep 17 00:00:00 2001
From: kerwizzy <kerwizzy@gmail.com>
Date: Wed, 9 Jan 2019 23:58:51 +0000
Subject: [PATCH] Mandelbrot: Clean up thread handling code.

 * Style fixes
 * Delete semaphores in destructor

Change-Id: Ia293d3f4096e2ef1305398356f3e263a8a263d78
Reviewed-on: https://review.haiku-os.org/c/866
Reviewed-by: waddlesplash <waddlesplash@gmail.com>
---
 src/apps/mandelbrot/FractalEngine.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/apps/mandelbrot/FractalEngine.cpp b/src/apps/mandelbrot/FractalEngine.cpp
index 447196bfd2..bdae067502 100644
--- a/src/apps/mandelbrot/FractalEngine.cpp
+++ b/src/apps/mandelbrot/FractalEngine.cpp
@@ -40,7 +40,7 @@ FractalEngine::FractalEngine(BHandler* parent, BLooper* looper)
 	fDoSet = &FractalEngine::DoSet_Mandelbrot;
 
 	fRenderSem = create_sem(0, "RenderSem");
-	fRenderStoppedSem = create_sem(0,"RenderStopped");
+	fRenderStoppedSem = create_sem(0, "RenderStopped");
 
 	system_info info;
 	get_system_info(&info);
@@ -60,6 +60,8 @@ FractalEngine::FractalEngine(BHandler* parent, BLooper* looper)
 
 FractalEngine::~FractalEngine()
 {
+	delete_sem(fRenderSem);
+	delete_sem(fRenderStoppedSem);
 }
 
 
@@ -158,6 +160,7 @@ void FractalEngine::WriteToBitmap(BBitmap* bitmap)
 		B_RGB24);
 }
 
+
 void FractalEngine::StopRender()
 {
 	if (fRenderStopped)
@@ -167,7 +170,7 @@ void FractalEngine::StopRender()
 		// the fRenderStoppedSem semaphores have already been acquired.
 	TRACE("Stopping render...\n");
 	fStopRender = true;
-	for (uint i = 0; i<fThreadCount; i++)
+	for (uint i = 0; i < fThreadCount; i++)
 		acquire_sem(fRenderStoppedSem);
 
 	TRACE("Render stopped.\n");