Changeset 3069 in Sophya for trunk/SophyaExt/FitsIOServer/fitshdtable.cc

Timestamp:

Sep 8, 2006, 4:30:31 PM (19 years ago)

Author:

ansari

Message:

Correction plusieurs bugs lies a SwFitsDataTable - Reza 08/09/2006

File:

• trunk/SophyaExt/FitsIOServer/fitshdtable.cc (modified) (4 diffs)

trunk/SophyaExt/FitsIOServer/fitshdtable.cc

@@ -188 +188 @@
 
   // Determination de la taille de segment 
-  sa_size_t segsz = is.GetNbRows() / 16; // Taille de segment par defaut 
+  sa_size_t segsz = is.GetNbRows() / 64; // Taille de segment par defaut 
   if (segsz > 2048) segsz = 2048;
   string key = "SEGMSIZE";
   bool knex = false;
   string ssegsz = is.KeyValue(key, knex);
-  if (!knex && (ssegsz.length() > 0))  segsz = atoi(ssegsz.c_str());
+  bool fgsss = false; 
+  if (!knex && (ssegsz.length() > 0))  { segsz = atoi(ssegsz.c_str());  fgsss = true; }
   if (segsz < 16) segsz = 16;
 
@@ -205 +206 @@
 
   if (dobj == NULL) { // On cree la table si besoin 
+    /*
+      Reza, Sep06 : On ne cree pas automatiquement un SwFitsDataTable - 
+      En effet, on ne peut garantir que l'objet FitsInOutFile is reste 
+      valide pendant toute la duree de vie de l'objet SwFitsDataTable
+
     string key = "SOPCLSNM"; 
     string clsnm = is.KeyValue(key);
-    if ( (clsnm == "SOPHYA::SwFitsDataTable") || (nbrows*colnames.size() < 10000000) )
+        if ( (clsnm == "SOPHYA::SwFitsDataTable") || (nbrows*colnames.size() < 10000000) )
       dobj = new DataTable(segsz);
-    else dobj = new SwFitsDataTable(is, segsz, false);
+    else { 
+      dobj = new SwFitsDataTable(segsz);
+      // dobj = new SwFitsDataTable(is, segsz, false);
+      // Reza, Sep2006: Ce constructeur de SwFitsDataTable lit l'entete 
+      // et initialise donc l'objet SwFitsDataTable  -> return
+      // return;
+    }
+    */
+    dobj = new DataTable(segsz);
   }
   else { 
     dobj->Clear();  // On efface la table sinon 
-    dobj->mSegSz = segsz;
+    if (fgsss)  dobj->mSegSz = segsz;
   }
 
@@ -221 +235 @@
 
   vector<int_8> swpos;
+  segsz = dobj->mSegSz;
+  int_8 swp = 1;
+  while (swp < nbrows) {
+    swpos.push_back(swp); swp += segsz;
+  }
   // Initialize the fits swap stream for SwFitsDataTable if necessary
   if (swfdt)  {
-    if ( swfdt->FitsSwapFile().FitsPtr() != is.FitsPtr() )
+    if ( swfdt->FitsSwapFile().FitsPtr() != is.FitsPtr() ) {
+      //      swfdt->FitsSwapFile().Close(); 
+      //      swfdt->FitsSwapFile().Open(is.FileName().c_str(), FitsInOutFile::Fits_RO);
+      //      swfdt->FitsSwapFile().MoveAbsToHDU(is.CurrentHDU());
       swfdt->FitsSwapFile().ShareFitsPtr(is); 
-    int_8 swp = 1;
-    
-    while (swp < nbrows) {
-      swpos.push_back(swp); swp += segsz;
     }
   }
@@ -303 +321 @@
 
   // On ne doit pas lire les donnees de la table pour un SwFitsDataTable
-  if (swfdt)  return;
+  if (swfdt) {
+    swfdt->mNSeg = swpos.size();
+    return;
+  }
 
   while ((dobj->SegmentSize()*dobj->NbSegments()) < dobj->NEntry())