compile in needed helpers and fix warnings

4 files changed, 16 insertions, 16 deletions

diff --git a/3rdparty/python-console/ColumnFormatter.cpp b/3rdparty/python-console/ColumnFormatter.cpp
index adbda96a..7e9d6712 100644
--- a/3rdparty/python-console/ColumnFormatter.cpp
+++ b/3rdparty/python-console/ColumnFormatter.cpp
@@ -52,9 +52,9 @@ int ColumnFormatter::solve(int width)
     while (fits)
     {
         ++i;
-        std::vector<int> widths = divideItems(i);
-        int columnWidth = width / i;
-        for (int j = 0; j < widths.size(); ++j)
+        std::vector<size_t> widths = divideItems(i);
+        size_t columnWidth = width / i;
+        for (size_t j = 0; j < widths.size(); ++j)
         {
             fits &= (widths[j] < columnWidth);
         }
@@ -66,18 +66,18 @@ int ColumnFormatter::solve(int width)
     return i;
 }
 
-std::vector<int> ColumnFormatter::divideItems(int numColumns)
+std::vector<size_t> ColumnFormatter::divideItems(int numColumns)
 {
     columns.clear();
     for (int i = 0; i < numColumns; ++i)
     columns.push_back(std::list<std::string>());
-    for (int i = 0; i < items.size(); ++i)
+    for (size_t i = 0; i < items.size(); ++i)
     {
         columns[i % numColumns].push_back(items[i]);
     }
 
     // count the fattest item in each column
-    std::vector<int> res(numColumns);
+    std::vector<size_t> res(numColumns);
     for (int i = 0; i < numColumns; ++i)
     {
         for (std::list<std::string>::const_iterator it =
@@ -104,7 +104,7 @@ void ColumnFormatter::format(int width)
     divideItems(cols);
     std::vector< std::list<std::string>::const_iterator > its;
     std::vector< std::list<std::string>::const_iterator > it_ends;
-    for (int i = 0; i < columns.size(); ++i)
+    for (size_t i = 0; i < columns.size(); ++i)
     {
         its.push_back(columns[i].begin());
         it_ends.push_back(columns[i].end());
@@ -113,7 +113,7 @@ void ColumnFormatter::format(int width)
     while (!done)
     {
         std::stringstream row_ss;
-        for (int i = 0; i < columns.size(); ++i)
+        for (size_t i = 0; i < columns.size(); ++i)
         {
             std::stringstream item_ss;
             std::string item;
@@ -128,7 +128,7 @@ void ColumnFormatter::format(int width)
         m_formattedOutput.push_back(row_ss.str());
 
         done = true;
-        for (int i = 0; i < columns.size(); ++i)
+        for (size_t i = 0; i < columns.size(); ++i)
         {
             done &= (its[i] == it_ends[i]);
         }
diff --git a/3rdparty/python-console/ColumnFormatter.h b/3rdparty/python-console/ColumnFormatter.h
index 4fedf2ea..f901b04a 100644
--- a/3rdparty/python-console/ColumnFormatter.h
+++ b/3rdparty/python-console/ColumnFormatter.h
@@ -60,7 +60,7 @@ public:
     /**
     Divide items into numColumns.
     */
-    std::vector<int> divideItems(int numColumns);
+    std::vector<size_t> divideItems(int numColumns);
 
     /**
     Generate formatted output, the items formatted into as many columns as can
diff --git a/3rdparty/python-console/ParseHelper.BracketParseState.cpp b/3rdparty/python-console/ParseHelper.BracketParseState.cpp
index 73694235..4ce41737 100644
--- a/3rdparty/python-console/ParseHelper.BracketParseState.cpp
+++ b/3rdparty/python-console/ParseHelper.BracketParseState.cpp
@@ -40,7 +40,7 @@ bool ParseHelper::BracketParseState::LoadBrackets(const std::string& str,
         return false;
 
     stack->clear();
-    for (int i = 0; i < str.size(); ++i)
+    for (size_t i = 0; i < str.size(); ++i)
     {
         if (OpeningBrackets.find_first_of(str[i]) != std::string::npos)
         {
@@ -63,7 +63,7 @@ bool ParseHelper::BracketParseState::LoadBrackets(const std::string& str,
 ParseHelper::BracketParseState::BracketParseState( ParseHelper& parent, const std::string& firstLine ):
     ParseState( parent )
 {
-    bool hasOpenBrackets = LoadBrackets( firstLine, &brackets );
+    /*bool hasOpenBrackets = */ LoadBrackets( firstLine, &brackets );
     //assert( hasOpenBrackets );
     m_buffer.push_back( firstLine );
 }
@@ -71,7 +71,7 @@ ParseHelper::BracketParseState::BracketParseState( ParseHelper& parent, const st
 bool ParseHelper::BracketParseState::process(const std::string& str)
 {
     // update brackets stack
-    for (int i = 0; i < str.size(); ++i)
+    for (size_t i = 0; i < str.size(); ++i)
     {
         if (OpeningBrackets.find_first_of(str[i]) != std::string::npos)
         {
diff --git a/3rdparty/python-console/ParseHelper.cpp b/3rdparty/python-console/ParseHelper.cpp
index 08f0526a..07616f5d 100644
--- a/3rdparty/python-console/ParseHelper.cpp
+++ b/3rdparty/python-console/ParseHelper.cpp
@@ -51,7 +51,7 @@ bool ParseHelper::PeekIndent( const std::string& str, Indent* indent )
         return false;
 
     int nonwhitespaceIndex = -1;
-    for (int i = 0; i < str.size(); ++i)
+    for (size_t i = 0; i < str.size(); ++i)
     {
         if (!isspace(str[i]))
         {
@@ -151,7 +151,7 @@ bool ParseHelper::buffered( ) const
 void ParseHelper::flush( )
 {
     std::stringstream ss;
-    for (int i = 0; i < commandBuffer.size(); ++i )
+    for (size_t i = 0; i < commandBuffer.size(); ++i )
     {
         ss << commandBuffer[i] << "\n";
     }
@@ -188,7 +188,7 @@ void ParseHelper::unsubscribeAll( )
 void ParseHelper::broadcast( const ParseMessage& msg )
 {
     // broadcast signal
-    for (int i = 0; i < listeners.size(); ++i)
+    for (size_t i = 0; i < listeners.size(); ++i)
     {
         if (listeners[i])
         {