Fix signed/unsigned comparisons.

diff --git a/common/trace_loader.cpp b/common/trace_loader.cpp
index c14d815382b4680b5b13eac008df33c828e6d8a7..51f3b556ea13a9d28bc59af9f749ec089a52df98 100644 (file)
--- a/common/trace_loader.cpp
+++ b/common/trace_loader.cpp
@@ -23,12 +23,12 @@ void Loader::setFrameMarker(Loader::FrameMarker marker)
     m_frameMarker = marker;
 }
 
-int Loader::numberOfFrames() const
+unsigned Loader::numberOfFrames() const
 {
     return m_frameBookmarks.size();
 }
 
-int Loader::numberOfCallsInFrame(int frameIdx) const
+unsigned Loader::numberOfCallsInFrame(unsigned frameIdx) const
 {
     if (frameIdx > m_frameBookmarks.size()) {
         return 0;
@@ -52,8 +52,8 @@ bool Loader::open(const char *filename)
 
     Trace::Call *call;
     ParseBookmark startBookmark;
-    int numOfFrames = 0;
-    int numOfCalls = 0;
+    unsigned numOfFrames = 0;
+    unsigned numOfCalls = 0;
     int lastPercentReport = 0;
 
     m_parser.getBookmark(startBookmark);
@@ -112,16 +112,16 @@ bool Loader::isCallAFrameMarker(const Trace::Call *call) const
     return false;
 }
 
-std::vector<Trace::Call *> Loader::frame(int idx)
+std::vector<Trace::Call *> Loader::frame(unsigned idx)
 {
-    int numOfCalls = numberOfCallsInFrame(idx);
+    unsigned numOfCalls = numberOfCallsInFrame(idx);
     if (numOfCalls) {
         const FrameBookmark &frameBookmark = m_frameBookmarks[idx];
         std::vector<Trace::Call*> calls(numOfCalls);
         m_parser.setBookmark(frameBookmark.start);
 
         Trace::Call *call;
-        int parsedCalls = 0;
+        unsigned parsedCalls = 0;
         while ((call = m_parser.parse_call())) {
 
             calls[parsedCalls] = call;

diff --git a/common/trace_loader.hpp b/common/trace_loader.hpp
index 9f74a9bde8e967d6de3e08a740bed1ce366a688b..61cc855826d8c161963d4c564c84aee61a231663 100644 (file)
--- a/common/trace_loader.hpp
+++ b/common/trace_loader.hpp
@@ -29,13 +29,13 @@ public:
     Loader::FrameMarker frameMarker() const;
     void setFrameMarker(Loader::FrameMarker marker);
 
-    int numberOfFrames() const;
-    int numberOfCallsInFrame(int frameIdx) const;
+    unsigned numberOfFrames() const;
+    unsigned numberOfCallsInFrame(unsigned frameIdx) const;
 
     bool open(const char *filename);
     void close();
 
-    std::vector<Trace::Call*> frame(int idx);
+    std::vector<Trace::Call*> frame(unsigned idx);
 
 private:
     struct FrameBookmark {
@@ -48,7 +48,7 @@ private:
         {}
 
         ParseBookmark start;
-        int numberOfCalls;
+        unsigned numberOfCalls;
     };
     bool isCallAFrameMarker(const Trace::Call *call) const;
 

diff --git a/common/trace_parser.cpp b/common/trace_parser.cpp
index 2eff0cda90bda893c9bbcf64e68c8c23158deff3..530c19a5a06fe00c7009a5cf7f7744971a0e41f1 100644 (file)
--- a/common/trace_parser.cpp
+++ b/common/trace_parser.cpp
@@ -228,7 +228,7 @@ FunctionSig *Parser::parse_function_sig(void) {
     } else if (file->currentOffset() < sig->offset) {
         /* skip over the signature */
         skip_string(); /* name */
-        int num_args = read_uint();
+        unsigned num_args = read_uint();
         for (unsigned i = 0; i < num_args; ++i) {
              skip_string(); /*arg_name*/
         }

diff --git a/common/trace_snappyfile.cpp b/common/trace_snappyfile.cpp
index 4dbe42dc364264a8db0e717a1af17681b9af271e..60711d2a5f2422648bcf4583b1b4d6b06e9782e3 100644 (file)
--- a/common/trace_snappyfile.cpp
+++ b/common/trace_snappyfile.cpp
@@ -122,18 +122,18 @@ bool SnappyFile::rawWrite(const void *buffer, size_t length)
         m_cachePtr += length;
         flushWriteCache();
     } else {
-        int sizeToWrite = length;
+        size_t sizeToWrite = length;
 
         while (sizeToWrite >= freeCacheSize()) {
-            int endSize = freeCacheSize();
-            int offset = length - sizeToWrite;
+            size_t endSize = freeCacheSize();
+            size_t offset = length - sizeToWrite;
             memcpy(m_cachePtr, (const char*)buffer + offset, endSize);
             sizeToWrite -= endSize;
             m_cachePtr += endSize;
             flushWriteCache();
         }
         if (sizeToWrite) {
-            int offset = length - sizeToWrite;
+            size_t offset = length - sizeToWrite;
             memcpy(m_cachePtr, (const char*)buffer + offset, sizeToWrite);
             m_cachePtr += sizeToWrite;
         }