[haiku-commits] haiku: hrev52739 - src/apps/mandelbrot

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 10 Jan 2019 00:27:45 -0500 (EST)

hrev52739 adds 1 changeset to branch 'master'
old head: f84c53e4acb4c8a9e4dadd4ce165ca3a126114b1
new head: cc15132ccc17de0e97006713692a23419e58f369
overview: 
https://git.haiku-os.org/haiku/log/?qt=range&q=cc15132ccc17+%5Ef84c53e4acb4

----------------------------------------------------------------------------

cc15132ccc17: 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@xxxxxxxxx>

                                           [ kerwizzy <kerwizzy@xxxxxxxxx> ]

----------------------------------------------------------------------------

Revision:    hrev52739
Commit:      cc15132ccc17de0e97006713692a23419e58f369
URL:         https://git.haiku-os.org/haiku/commit/?id=cc15132ccc17
Author:      kerwizzy <kerwizzy@xxxxxxxxx>
Date:        Wed Jan  9 23:58:51 2019 UTC
Committer:   waddlesplash <waddlesplash@xxxxxxxxx>
Commit-Date: Thu Jan 10 05:27:42 2019 UTC

----------------------------------------------------------------------------

1 file changed, 5 insertions(+), 2 deletions(-)
src/apps/mandelbrot/FractalEngine.cpp | 7 +++++--

----------------------------------------------------------------------------

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");


Other related posts:

  • » [haiku-commits] haiku: hrev52739 - src/apps/mandelbrot - waddlesplash