Fixed more compilation warnings
authorTito Dal Canton <tito.canton@ligo.org>
Fri, 25 Jan 2013 10:52:39 +0000 (11:52 +0100)
committerTito Dal Canton <tito.canton@ligo.org>
Fri, 25 Jan 2013 10:52:39 +0000 (11:52 +0100)
common/include/array/GWArray.hpp
common/include/chi2/GWChi2.hpp
common/include/filter/GWFilter.hpp
common/include/series/GWSeries.hpp
common/include/templatebank/GWTemplateBank.hpp
common/include/waveform/GWWaveformTaylorF2.hpp

index c4f65b6..430dcab 100644 (file)
@@ -1211,7 +1211,7 @@ template <typename T> cl_int Array <T>::Write(const char * filename, const char
                myFile.close();
        } else if (std::string(format) == "ascii") {
                ofstream myFile (filename, ios::out | ios::trunc);
-               for (int i = 0; i < length; i++) {
+               for (unsigned int i = 0; i < length; i++) {
                        myFile << i << " " << myBuff[i] << endl;
                }
                myFile.close();
@@ -1229,7 +1229,7 @@ template <typename T> cl_int Array <T>::Read(const char * filename, const char *
                myFile.close();
        } else if (std::string(format) == "ascii") {
                ifstream myFile (filename, ios::in );
-        for (int i = 0; i < length; i++) {
+        for (unsigned int i = 0; i < length; i++) {
             myFile >> myBuff[i];
         }
         myFile.close();
index 81a6da6..7ae46e0 100644 (file)
@@ -197,7 +197,7 @@ Chi2<T>::CalculateChi2Series(       cl_uint number_of_bands,
        }
 
        // Coping the frequency bands to a separate buffer
-       for (int k = 0 ; k < number_of_bands ; k ++) {
+       for (unsigned int k = 0 ; k < number_of_bands ; k ++) {
                templ->Copy(bands[k], k * templ->length + bands[k], bands[k + 1] - bands[k], work_space);
        }
        aux.s[0] = 0;
index a7ec957..56c9179 100644 (file)
@@ -350,7 +350,7 @@ template <typename T> int Filter<T>::LowPass(TimeSeries <T> * src, double freq,
                                                               freq, half_width);
     // Calculating the necessary indexes
     double df        = 1. / (src->delta_t * src->length);
-    cl_int idx       = freq / df;
+    cl_uint idx      = freq / df;
     cl_int idx_width = half_width / df;
 
     // Converting to Fourier domain.
@@ -363,7 +363,7 @@ template <typename T> int Filter<T>::LowPass(TimeSeries <T> * src, double freq,
     }
 
     // Converting back to time domain and normalizing
-    myFFT->Inverse(src,src);
+    myFFT->Inverse(src, src);
     *src /= N;
     return CL_SUCCESS;
 }
@@ -615,7 +615,7 @@ template <typename T> int Filter<T>::PSD(TimeSeries <T> * myArray,
                 values.push_back(seg_buf[i * segment_size + j].real());
             }
             std::sort(values.begin(), values.end());
-            if (number_of_segments & 1 == 0)
+            if ((number_of_segments & 1) == 0)
                 median_buf[j] = 0.5 * (values[number_of_segments / 2 - 1] + values[number_of_segments / 2]);
             else
                 median_buf[j] = values[number_of_segments / 2];
index 2cb90e7..d4ebd18 100644 (file)
@@ -133,9 +133,8 @@ template <typename T> cl_int TimeSeries <T>::Segment ( std::vector <TimeSeries <
                                                         cl_uint segment_length,
                                                         cl_uint overlap)
 {
-    cl_int  err          = 0;
     cl_uint num_segments = (length - segment_length) / ( segment_length - overlap ) + 1;
-    for (int i = 0; i < num_segments; i++) {
+    for (unsigned int i = 0; i < num_segments; i++) {
         dest.push_back(TimeSeries <T> ( segment_length, delta_t, m_commif, m_logger, m_runtime));
         Copy(i * (segment_length - overlap), 0, segment_length, &dest[i]);
     }
index c8461be..059597b 100644 (file)
@@ -106,7 +106,7 @@ template  <typename T> void TemplateBank<T>::GetLongestTemplate(
        m_logger->Log(GW::DEBUG,__func__,
                  "Looping through on the %d bank templates.", templates.size());
 
-    for (int i = 0; i < templates.size(); i++) {
+    for (unsigned int i = 0; i < templates.size(); i++) {
         length = mywaveform->Duration(templates[i]);
         if (length > maxlength) {
             maxlength = length;
@@ -131,6 +131,8 @@ template <typename T> cl_int TemplateBank<T>::Read(const std::string filename)
     else if (boost::algorithm::ends_with(filename, ".xml") ||
             boost::algorithm::ends_with(filename, ".xml.gz"))
         return ReadFromMetaIO(filename);
+    else
+        return -1;
 }
 
 /******************************************************************************/
index ad003e7..9a19a37 100644 (file)
@@ -206,7 +206,7 @@ template <typename T>  int TaylorF2<T>::Generate(
     std::vector <string> out;
     boost::split(out, output, boost::is_any_of(", "), boost::token_compress_on);
 
-    for (int i = 0; i < out.size(); i++) {
+    for (unsigned int i = 0; i < out.size(); i++) {
         switch(outputMap[out[i]])
         {
           case o_h_sin: