Changeset 2055

Timestamp:

Sep 8, 2009, 8:23:43 PM (14 years ago)

Author:

Peter

Message:

fixed some warnings from gcc 4.3.1 and added some missing includes. There are still some warnings for test/commandline_test.cc, but not obvious how to fix them.

Location:

trunk

Files:

• test/Suite.cc (modified) (1 diff)
• test/data_lookup_1d_test.cc (modified) (1 diff)
• test/fileutil_test.cc (modified) (2 diffs)
• test/large_file_test.cc (modified) (1 diff)
• yat/classifier/KNN.h (modified) (1 diff)
• yat/classifier/SvmMultiClass.cc (modified) (1 diff)
• yat/normalizer/qQuantileNormalizer.h (modified) (1 diff)
• yat/statistics/FoldChange.h (modified) (1 diff)
• yat/statistics/KolmogorovSmirnov.h (modified) (1 diff)
• yat/statistics/utility.cc (modified) (1 diff)
• yat/utility/FileUtil.cc (modified) (1 diff)
• yat/utility/OptionFile.cc (modified) (1 diff)
• yat/utility/OptionHelp.cc (modified) (1 diff)
• yat/utility/utility.h (modified) (2 diffs)

trunk/test/Suite.cc

@@ -28 +28 @@
 #include <fstream>
 #include <iostream>
+#include <limits>
 #include <sstream>
 #include <string>

trunk/test/data_lookup_1d_test.cc

@@ -32 +32 @@
 #include <fstream>
 #include <iostream>
+#include <limits>
 #include <vector>
 #include <cmath>

trunk/test/fileutil_test.cc

@@ -47 +47 @@
     bool testval=true;
     utility::FileUtil file(".");
-    if (testval=file.exists())
+    if ((testval=file.exists()))
       suite.err() << "\tfile exists: " << file.path() << std::endl;
     suite.add(testval);
@@ -133 +133 @@
       suite.err() << "\tr permission failed on " << file.path() << std::endl;
     suite.add(testval);
-    if (testval=file.permissions("rwx"))
+    if ((testval=file.permissions("rwx")))
       suite.err() << "\trwx permissions failed on " << file.path() << std::endl;
     suite.add(testval);
-    if (testval=file.permissions("w"))
+    if ((testval=file.permissions("w")))
       suite.err() << "\tw permission failed on " << file.path() << std::endl;
     suite.add(testval);
-    if (testval=file.permissions("d"))
+    if ((testval=file.permissions("d")))
       suite.err() << "\td permission failed on " << file.path() << std::endl;
     suite.add(testval); 

trunk/test/large_file_test.cc

@@ -27 +27 @@
 #include "Suite.h"
 
+#include <limits>
 #include <string>
 #include <sys/stat.h>

trunk/yat/classifier/KNN.h

@@ -35 +35 @@
 
 #include <cmath>
+#include <limits>
 #include <map>
 #include <stdexcept>

trunk/yat/classifier/SvmMultiClass.cc

@@ -28 +28 @@
 
 #include <cassert>
-#include <climits>
+#include <limits>
 
 namespace theplu {

trunk/yat/normalizer/qQuantileNormalizer.h

@@ -33 +33 @@
 #include <cmath>
 #include <iterator>
+#include <limits>
 #include <numeric>
 #include <stdexcept>

trunk/yat/statistics/FoldChange.h

@@ -28 +28 @@
 namespace theplu {
 namespace yat {
-  class utility::VectorBase;
+namespace utility {
+  class VectorBase;
+}
 namespace statistics {
 

trunk/yat/statistics/KolmogorovSmirnov.h

@@ -24 +24 @@
 */
 
+#include <iosfwd>
 #include <set>
 #include <vector>

trunk/yat/statistics/utility.cc

@@ -31 +31 @@
 #include <algorithm>
 #include <cassert>
+#include <limits>
 
 namespace theplu {

trunk/yat/utility/FileUtil.cc

@@ -33 +33 @@
 #include <cerrno>
 #include <cstddef>
+#include <cstring>
 #include <iostream>
 #include <stdexcept>

trunk/yat/utility/OptionFile.cc

@@ -25 +25 @@
 #include "FileUtil.h"
 
+#include <cstring>
 #include <string>
 

trunk/yat/utility/OptionHelp.cc

@@ -23 +23 @@
 #include "CommandLine.h"
 
+#include <cstdlib>
 #include <iostream>
 #include <string>

trunk/yat/utility/utility.h

@@ -205 +205 @@
     if (is_equal(s, "inf"))
       return std::numeric_limits<T>::infinity();
-    if (is_equal(s, "-inf"))
+    if (is_equal(s, "-inf")) {
       if (std::numeric_limits<T>::is_signed)
         return -std::numeric_limits<T>::infinity();
@@ -211 +211 @@
         throw std::runtime_error(std::string("yat::utility::convert(\"")+s+
                                  std::string("\"): type is unsigned") );
+    }
     std::stringstream ss(s);
     T a;