Clean code

[creaImageIO.git] / src / creaImageIOWxViewer.cpp

diff --git a/src/creaImageIOWxViewer.cpp b/src/creaImageIOWxViewer.cpp
index ee2258aa2ea73d8b4b921e1e3dba275799a261dd..22e1b3dbe2c6fd919a28f4af747f0a42708f535e 100644 (file)
--- a/src/creaImageIOWxViewer.cpp
+++ b/src/creaImageIOWxViewer.cpp
@@ -43,7 +43,22 @@ namespace creaImageIO
   
   //=====================================================================
 
+  class wxThreadEED3
+  {
+  public:
+         void Delete() {  printf("EED wxThreadEED3::Delete() \n"); }
+         int Run() { printf("EED wxThreadEED3::Run() \n"); return 0;}
+         void Pause() { printf("EED wxThreadEED3::Pause() \n"); }
+         void Create() { printf("EED wxThreadEED3::Create() \n"); }
+         bool IsAlive() { printf("EED wxThreadEED3::IsAlive() \n"); return false; }
+         bool TestDestroy() { printf("EED wxThreadEED3::TestDestroy() \n"); return false; }
+         int GetCurrentId() { printf("EED wxThreadEED3::GetCurrentId() \n"); return -999; }      
+  };
+
+  
   //=====================================================================
+//EED 2018-08-20
+//  class WxViewerPlayer: public wxThreadEED3
   class WxViewerPlayer: public wxThread
   {
   public:
@@ -78,10 +93,9 @@ namespace creaImageIO
                 size)
   {
     wxMutexLocker lock(mMutex);
-    GimmickDebugMessage(6,"WxViewer::WxViewer"
-                       <<std::endl);
+    GimmickDebugMessage(6,"WxViewer::WxViewer" <<std::endl);
 
-    mNeedRefresh = false;
+    mNeedRefresh       = false;
     mLastImageShown = NULL;
        
        // previewer    
@@ -91,8 +105,8 @@ namespace creaImageIO
     mViewer    = vtkImageViewer2::New();
     mViewer->SetupInteractor ( mInteractor );
     
-    mCurrent = 0;
-       mPlayer = 0;
+    mCurrent   = 0;
+       mPlayer         = NULL;
 
        // Grid to place checkbox and slider 
        mflexSizer = new wxFlexGridSizer(1,2,1,1);
@@ -122,8 +136,7 @@ namespace creaImageIO
   WxViewer::~WxViewer()
   {
     wxMutexLocker lock(mMutex);
-    GimmickDebugMessage(6,"WxViewer::~WxViewer"
-                       <<std::endl);
+    GimmickDebugMessage(6,"WxViewer::~WxViewer"<<std::endl);
     // TO DO : desallocate cleanly
        if(mPlayer)
        {
@@ -158,9 +171,7 @@ namespace creaImageIO
 
   void WxViewer::ShowNextImage()
   {
-        
-       
-         mMutex.Unlock();
+       mMutex.Unlock();
        wxMutexLocker lock(mMutex);
            
     GimmickMessage(2,"WxViewer::ShowNextImage() "
@@ -180,10 +191,8 @@ namespace creaImageIO
                                mNeedRefresh = true;
                                mLastImageShown = currIm;
                        }
-               mCurrent++;
-           }
-               else
-               {
+                       mCurrent++;
+           } else {
                        mCurrent = 0;
                        //ImagePointerHolder* iph=imagePointers[mCurrent];
                        boost::shared_ptr<ImagePointerHolder> iph = imagePointers[mCurrent];
@@ -195,8 +204,8 @@ namespace creaImageIO
                                mLastImageShown = currIm;
                        }
                        mCurrent++;
-               }
-       }
+               } // if mCurrent
+       } // if imagePointers
   }
   //================================================================
 
@@ -299,14 +308,13 @@ std::cout << std::endl;
   //================================================================
   bool WxViewer::RefreshIfNecessary()
   {
-    if (mNeedRefresh)
-      {
-       GimmickDebugMessage(10,"WxViewer : Refreshing"<<std::endl);
-
-       mInteractor->Render();
-       mNeedRefresh = false;
-       return true;
-      }
+    if (mNeedRefresh==true)
+    {
+               GimmickDebugMessage(10,"WxViewer : Refreshing"<<std::endl);
+               mInteractor->Render();
+               mNeedRefresh = false;
+               return true;
+    }
     return false;
   }
   //================================================================
@@ -315,9 +323,9 @@ std::cout << std::endl;
   void WxViewer::StopPlayer()
   {
          wxMutexLocker lock(mMutex);
-         if (mPlayer==0 ) return;
+         if (mPlayer==NULL ) return;
          mPlayer->Delete();  
-         mPlayer=0;
+         mPlayer=NULL;
          mMutex.Unlock();
   }
   //================================================================
@@ -328,15 +336,13 @@ std::cout << std::endl;
                if(mcheck->IsChecked())
                {
                        //      wxMutexLocker lock(mMutex);
-                       if (mPlayer != 0) return;
+                       if (mPlayer != NULL) return;
                        mPlayer = new WxViewerPlayer(this);
                        mPlayer->Create();
                        mPlayer->Run();  
-               }
-               else
-               {
+               } else {
                        ShowNextImage();
-               }
+               } // if
        }
 
   //================================================================
@@ -349,7 +355,7 @@ std::cout << std::endl;
                {
                        mPlayer->Pause();
                        mPlayer->Delete();
-                       mPlayer = 0;
+                       mPlayer = NULL;
                }
                StartPlayer();
        }