Changeset 12724 for trunk

Timestamp:

12/15/11 10:47:08 (13 years ago)

Author:

lyard

Message:

fixed fitsdump warnings

File:

• trunk/FACT++/src/fitsdump.cc (modified) (7 diffs)

trunk/FACT++/src/fitsdump.cc

@@ -13 +13 @@
 #include "externals/fits.h"
 
-#define PLOTTING_PLEASE
+//#define PLOTTING_PLEASE
 
 #ifdef PLOTTING_PLEASE
@@ -57 +57 @@
 
 private:
+    fits* fFile;
     bool fDotsPlease;
-    fits* fFile;
     string fFilename;
 
@@ -205 +205 @@
 }
 
-bool FitsDumper::OpenTable(const string &tablename)
+bool FitsDumper::OpenTable(const string &)
 {
     if (!fFile)
@@ -370 +370 @@
         const fits::Table::Column& col = fColMap[*it];
 //        const MyColumn *col = static_cast<MyColumn*>(fTable->column()[*it]);
-        if (rangesIt->first != 0 || rangesIt->second != col.num)
+        if (rangesIt->first != 0 || rangesIt->second != (int)(col.num))
         {
             out << "#";
@@ -632 +632 @@
     rangesIt = ranges.begin();
     auto statsIt = statData.begin();
-    int round;
+
     auto nameIt = listNamesOnly.begin();
     for (auto it=columnsData.begin(); it != columnsData.end(); it++, rangesIt++, statsIt++, nameIt++)
@@ -785 +785 @@
 //      stringstream str;
       str.str("");
-      int status = 0;
+
 
       vector<double*> xValues(totalSize);
@@ -807 +807 @@
           for (auto it=columnsData.begin(); it != columnsData.end(); it++, rangesIt++)
           {
-              for (int i=rangesIt->first; i<rangesIt->second; i++)
+              for (int j=rangesIt->first; j<rangesIt->second; j++)
               {
               switch (it->first) {
                   case 'L':
-                          str << reinterpret_cast<bool*>(it->second)[i] << " ";
+                          str << reinterpret_cast<bool*>(it->second)[j] << " ";
                           break;
                   case 'B':
-                          str << reinterpret_cast<char*>(it->second)[i] << " ";
+                          str << reinterpret_cast<char*>(it->second)[j] << " ";
                           break;
                   case 'I':
-                          str << reinterpret_cast<int16_t*>(it->second)[i] << " ";
+                          str << reinterpret_cast<int16_t*>(it->second)[j] << " ";
                           break;
                   case 'J':
-                          str << reinterpret_cast<int32_t*>(it->second)[i] << " ";
+                          str << reinterpret_cast<int32_t*>(it->second)[j] << " ";
                           break;
                   case 'K':
-                          str << reinterpret_cast<int64_t*>(it->second)[i] << " ";
+                          str << reinterpret_cast<int64_t*>(it->second)[j] << " ";
                           break;
                   case 'E':
-                          str << reinterpret_cast<float*>(it->second)[i] << " ";
+                          str << reinterpret_cast<float*>(it->second)[j] << " ";
                           break;
                   case 'D':
-                          str << reinterpret_cast<double*>(it->second)[i] << " ";
+                          str << reinterpret_cast<double*>(it->second)[j] << " ";
                           break;
                   default: