From: smh <smh@opencascade.com>
Date: Thu, 22 Apr 2004 07:23:55 +0000 (+0000)
Subject: Remove unused parameters and methods
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=cdf47faa1a94c1359c7acc1045aabc4d7c624160;p=modules%2Fkernel.git

Remove unused parameters and methods
---

diff --git a/src/Event/SALOME_Event.cxx b/src/Event/SALOME_Event.cxx
index 7de746341..dfebd7b02 100644
--- a/src/Event/SALOME_Event.cxx
+++ b/src/Event/SALOME_Event.cxx
@@ -46,10 +46,9 @@ static int MYDEBUG = 0;
  *  Constructor
  */
 //===========================================================
-SALOME_Event::SALOME_Event( int salomeEventType, bool wait, bool autoRelease ): 
-  myType( salomeEventType ), 
-  myWait( wait ),
-  myAutoRelease( autoRelease )
+SALOME_Event::SALOME_Event(): 
+  myWait( true ),
+  myAutoRelease( false )
 {
   if(MYDEBUG) MESSAGE( "SALOME_Event::SALOME_Event(): this = "<<this<<", myWait = "<<myWait );
   if ( myWait ) {
diff --git a/src/Event/SALOME_Event.hxx b/src/Event/SALOME_Event.hxx
index 198d1d63c..2467912d9 100644
--- a/src/Event/SALOME_Event.hxx
+++ b/src/Event/SALOME_Event.hxx
@@ -85,12 +85,11 @@ class QSemaphore;
 class SALOME_Event
 {
 public:
-  SALOME_Event( int salomeEventType = -1, bool wait = true, bool autoRelease = false );
+  SALOME_Event();
   virtual ~SALOME_Event();
 
-  int getType() const { return myType; }
-
-  virtual bool Execute() { return false; }
+  // To do real work
+  virtual void Execute() = 0;
 
   void process();
   void processed();
@@ -98,7 +97,6 @@ public:
   void release();
 
 private:
-  int         myType;
   bool        myWait;
   bool        myAutoRelease;
   QSemaphore* mySemaphore;
@@ -119,9 +117,8 @@ public:
     myAction(theAction),
     myResult(theResult)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     myResult = (myObject->*myAction)();
-    return true; 
   }
 private:
   TObject* myObject;
@@ -137,9 +134,8 @@ public:
     myObject(theObject),
     myAction(theAction)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     (myObject->*myAction)();
-    return true; 
   }
 private:
   TObject* myObject;
@@ -163,9 +159,8 @@ public:
     myResult(theResult),
     myArg(theArg)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     myResult = (myObject->*myAction)(myArg);
-    return true; 
   }
 private:
   TObject* myObject;
@@ -183,9 +178,8 @@ public:
     myAction(theAction),
     myArg(theArg)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     (myObject->*myAction)(myArg);
-    return true; 
   }
 private:
   TObject* myObject;
@@ -213,9 +207,8 @@ public:
     myArg(theArg),
     myArg1(theArg1)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     myResult = (myObject->*myAction)(myArg,myArg1);
-    return true; 
   }
 private:
   TObject* myObject;
@@ -236,9 +229,8 @@ public:
     myArg(theArg),
     myArg1(theArg1)
   {}
-  virtual bool Execute(){
+  virtual void Execute(){
     (myObject->*myAction)(myArg,myArg1);
-    return true; 
   }
 private:
   TObject* myObject;
@@ -263,4 +255,5 @@ inline void ProcessVoidEvent(SALOME_Event* theEvent){
   theEvent->release();
 }
 
+
 #endif
diff --git a/src/SALOMEGUI/QAD_Desktop.cxx b/src/SALOMEGUI/QAD_Desktop.cxx
index 892cc2e52..14b525a88 100644
--- a/src/SALOMEGUI/QAD_Desktop.cxx
+++ b/src/SALOMEGUI/QAD_Desktop.cxx
@@ -452,15 +452,9 @@ bool QAD_Desktop::eventFilter( QObject* o, QEvent* e )
   }
   else if ( e->type() == SALOME_EVENT ) { 
     SALOME_Event* aSE = (SALOME_Event*)((QCustomEvent*)e)->data();
-    MESSAGE( "QAD_Desktop::eventFilter - SALOME_Event handling - 1 : o = " << o << ", e = " << aSE);
-
     processEvent( aSE );
-    MESSAGE( "QAD_Desktop::eventFilter - SALOME_Event handling - 2" );
-
     // Signal the calling thread that the event has been processed
     aSE->processed();
-    MESSAGE( "QAD_Desktop::eventFilter - SALOME_Event handling - 3" );
-
     ((QCustomEvent*)e)->setData( 0 );
     delete aSE;
     return TRUE;
@@ -475,16 +469,7 @@ void QAD_Desktop::processEvent( SALOME_Event* theEvent )
 {
   if ( !theEvent )
     return;
-
-  if(theEvent->Execute())
-    return;
-
-  // san - temporary code - to be removed together with VISU_Event and test operations in VISU_I...
-  for ( ComponentMap::iterator it = myComponents.begin(); it != myComponents.end(); it++ ) {
-    if ( it.data()->CanProcessEvent( theEvent ) && it.data()->ProcessEvent( theEvent ) )
-      break;
-  }
-  // san - temporary code - to be removed together with VISU_Event and test operations in VISU_I...
+  theEvent->Execute();
 }
 
 /*!
diff --git a/src/SALOMEGUI/SALOMEGUI.cxx b/src/SALOMEGUI/SALOMEGUI.cxx
index e3843bd25..d0abefe31 100644
--- a/src/SALOMEGUI/SALOMEGUI.cxx
+++ b/src/SALOMEGUI/SALOMEGUI.cxx
@@ -58,26 +58,6 @@ bool SALOMEGUI::OnGUIEvent(int theCommandID, QAD_Desktop* parent)
   return true;
 }
 
-//=============================================================================
-/*!
- *
- */
-//=============================================================================
-bool SALOMEGUI::CanProcessEvent(SALOME_Event* se)
-{
-  return false;
-}
-
-//=============================================================================
-/*!
- *
- */
-//=============================================================================
-bool SALOMEGUI::ProcessEvent(SALOME_Event* se)
-{
-  return false;
-}
-
 //=============================================================================
 /*!
  *
diff --git a/src/SALOMEGUI/SALOMEGUI.h b/src/SALOMEGUI/SALOMEGUI.h
index 0594487d4..124ceee0b 100644
--- a/src/SALOMEGUI/SALOMEGUI.h
+++ b/src/SALOMEGUI/SALOMEGUI.h
@@ -55,8 +55,6 @@ public:
   virtual ~SALOMEGUI();
 
   virtual bool OnGUIEvent        (int theCommandID, QAD_Desktop* parent);
-  virtual bool CanProcessEvent   (SALOME_Event* se);
-  virtual bool ProcessEvent      (SALOME_Event* se);
   virtual bool OnKeyPress        (QKeyEvent* pe, QAD_Desktop* parent, QAD_StudyFrame* studyFrame);
   virtual bool OnMousePress      (QMouseEvent* pe, QAD_Desktop* parent, QAD_StudyFrame* studyFrame);
   virtual bool OnMouseMove       (QMouseEvent* pe, QAD_Desktop* parent, QAD_StudyFrame* studyFrame);