Changeset 2776 in Sophya

Timestamp:

May 27, 2005, 5:08:16 PM (20 years ago)

Author:

ansari

Message:

Tentative de correction d'appel au mauvais FinishCreate() - Reza 27/5/2005

Location:

trunk/SophyaPI/PI

Files:

• pibwdgx.cc (modified) (5 diffs)
• pibwdgx.h (modified) (1 diff)
• picontainergen.cc (modified) (3 diffs)
• picontainerx.cc (modified) (2 diffs)
• piimage.cc (modified) (3 diffs)
• pioptmenux.cc (modified) (1 diff)
• piwdgx.cc (modified) (1 diff)

trunk/SophyaPI/PI/pibwdgx.cc

@@ -29 +29 @@
 PIBaseWdgX::PIBaseWdgX(PIContainerGen *par, const char *nom,
                        int sx, int sy, int px, int py) 
-: PIBaseWdgGen(par, nom, sx, sy, px, py)
+  : PIBaseWdgGen(par, nom, sx, sy, px, py) , bwx_finishDone(false)
 
 {
@@ -36 +36 @@
 Init();
 mWGrC = NULL; // PIGraphic (Contexte graphique) associe a la fenetre
-finishDone = false;     // FinishCreate() pas encore appele
 
 CreateXtWdg(nom, piXtBaseWidgetClass, NULL, sx, sy, px, py);
@@ -50 +49 @@
 mPtrS = PI_CrossPointer;
 SelPointerShape();
-// PIGraphic (Contexte graphique) associe a la fenetre
+// ----------------------------------------------------------------------------
+// Reza: Fev/Mai 2005 - On doit s'assurer que le bon FinishCreate() est appele
+// (table de fonction virtuelle mise a jour par les constructeurs successifs 
+// dans les classes heritieres
+// et on gere le PIGraphic (Contexte graphique) associe a la fenetre
 // Si pas cree par FinishCreate()
-if (!mWGrC) mWGrC = new PIGraphicX(this);   
+if (finishDone) FinishCreate();
+else mWGrC = new PIGraphicX(this);   
+
 SetBackgroundColor();
 SetForegroundColor();
@@ -75 +80 @@
 {
 // appele par la fenetre mere, apres XtRealize()
-if (finishDone) { 
-  fprintf(stderr, "BUG/PIBaseWdgX::FinishCreate() multiple call \n"); 
-  return; 
-  }
+// Doit pouvoir etre appele de multiple fois 
+if (bwx_finishDone) return;
 PIWdgX::FinishCreate();
 
@@ -101 +104 @@
 if (!mWGrC) mWGrC = new PIGraphicX(this);   
 mWGrC->TerminateInit();
+bwx_finishDone = true;
 } 
 

trunk/SophyaPI/PI/pibwdgx.h

@@ -52 +52 @@
   void               Init();
   PIPointer mPtrS;
+// pour s'assurer que le bon FinishCreate() est appele - pb lie aux tables de fonctions
+// virtuelles des classes qui sont remplies au fur et a mesure des constructeurs
+  bool               bwx_finishDone; 
 };
 

trunk/SophyaPI/PI/picontainergen.cc

@@ -46 +46 @@
 //      Création d'un objet "PIContainer" dans le conteneur parent "par", de
 //      taille "sx,sy", se trouvant à la position "px,py" avec le nom "nom".
-//      Tous les widgets fils sont détruits (delete) par le destructeur
-//      (voir "SetAutoDelChilds()").
-//      Par défaut, le conteneur créé est visible (actif).
+//      Peut controler la destruction des widgets fils (voir "SetAutoDelChilds()").
+//      Le conteneur créé n'est pas visible (actif) par défaut. Il faut 
+//      appeler "Show()".
 // long kind()
 //      Renvoie le type de la classe de l'objet ("=PIContainerGen::ClassId") pour cette classe.
@@ -142 +142 @@
 void PIContainerGen::Show()
 {
+Manage();
 stvis = true;
-Manage();
-return;
+FinishCreate();
 }
 
@@ -200 +200 @@
 void PIContainerGen::FinishCreate()
 {
+if (!stvis) return;
 if (stfcr)  return;
 PIWdg::FinishCreate();    
-int k;
-for(k=0; k<NbChilds(); k++)  GetChild(k)->FinishCreate();
+for(int k=0; k<NbChilds(); k++)  GetChild(k)->FinishCreate();
 stfcr = true;
 }

trunk/SophyaPI/PI/picontainerx.cc

@@ -33 +33 @@
 // Pour gerer le probleme de l'appel a FinishCreate() 
 // lorsque celui du parent est deja appele 
-if (finishDone)  {
-  //DBG  cout << " PIContainerX::PIContainerX/DBG - finishDone - Nom= " << nom << endl;
-  Manage();
-  stfcr = true;
-  }
+if (finishDone)  FinishCreate(); 
 }
 
@@ -59 +55 @@
 // *************** Reza - Fev 2005 ***************
 // Probleme lie a la table des fonctions virtuelles - voir ci-dessus
-if (finishDone)  {
-  //DBG  cout << " PIContainerX::PIContainerX_Special/DBG - finishDone - Nom= " << nom << endl;
-  Manage();
-  stfcr = true;
-  }
+if (finishDone)  FinishCreate();
 }
 

trunk/SophyaPI/PI/piimage.cc

@@ -9 +9 @@
 #include <stdlib.h>
 #include <stdio.h>
+#include <iostream>
 #include <string.h>
 #include <ctype.h>
@@ -1619 +1620 @@
   piimg->SetGloVWin(gimv, false);
   piimg->SetCMapWin(cmapv, false);
-    
 }
 
@@ -1625 +1625 @@
 PIImageNavigator::~PIImageNavigator()
 {
+  if (piimg) delete piimg;
+  if (zoom)  delete zoom;
+  if (gimv)  delete gimv;
+  if (cmapv) delete cmapv;
 }
 

trunk/SophyaPI/PI/pioptmenux.cc

@@ -63 +63 @@
 mMen->SetMsgParent(this);
 Manage();
+if (finishDone) FinishCreate();
 }
 

trunk/SophyaPI/PI/piwdgx.cc

@@ -223 +223 @@
 {
 // appele par la fenetre mere, apres XtRealize()
-if (finishDone) { 
-  fprintf(stderr, "BUG/PIWdgX::FinishCreate() multiple call \n"); 
-  return; 
-  }
+// Doit pouvoir etre appele plusieurs fois
 finishDone = true;
 return;