size_t fixes, thanks Semushin Slava aka php-coder

10 files changed, 15 insertions, 14 deletions

diff --git a/src/FbTk/ImageControl.cc b/src/FbTk/ImageControl.cc
index 902bd55..7fe9154 100644
--- a/src/FbTk/ImageControl.cc
+++ b/src/FbTk/ImageControl.cc
@@ -258,7 +258,7 @@ Pixmap ImageControl::renderImage(unsigned int width, unsigned int height,
 
         cache.push_back(tmp);
 
-        if ((unsigned) cache.size() > cache_max)
+        if (cache.size() > cache_max)
             cleanCache();
 
         return pixmap;
diff --git a/src/FbTk/Layer.hh b/src/FbTk/Layer.hh
index 3e206b2..5c0c566 100644
--- a/src/FbTk/Layer.hh
+++ b/src/FbTk/Layer.hh
@@ -59,7 +59,7 @@ public:
     /// @return layer item on specific position, on failure 0
     ItemType *getItem(unsigned int position);
     /// @return number of elements in layer
-    unsigned int size() const { return m_list.size(); }
+    size_t size() const { return m_list.size(); }
     /// @return layer list
     const ListType &itemList() const { return m_list; }
     /// @return layer list
diff --git a/src/FbTk/MacroCommand.cc b/src/FbTk/MacroCommand.cc
index 84794e1..adee9cb 100644
--- a/src/FbTk/MacroCommand.cc
+++ b/src/FbTk/MacroCommand.cc
@@ -29,7 +29,7 @@ void MacroCommand::add(RefCount<Command> &com) {
     m_commandlist.push_back(com);
 }
 
-unsigned int MacroCommand::size() const {
+size_t MacroCommand::size() const {
     return m_commandlist.size();
 }
 
diff --git a/src/FbTk/MacroCommand.hh b/src/FbTk/MacroCommand.hh
index 97cc5d5..f29e05c 100644
--- a/src/FbTk/MacroCommand.hh
+++ b/src/FbTk/MacroCommand.hh
@@ -35,7 +35,7 @@ namespace FbTk {
 class MacroCommand:public Command {
 public:
     void add(RefCount<Command> &com);
-    unsigned int size() const;
+    size_t size() const;
     virtual void execute();
 
 private:
diff --git a/src/FbTk/Menu.cc b/src/FbTk/Menu.cc
index 5687be1..0616906 100644
--- a/src/FbTk/Menu.cc
+++ b/src/FbTk/Menu.cc
@@ -577,13 +577,13 @@ void Menu::clearWindow() {
     menu.frame.clear();
     
     // clear foreground bits of frame items
-    for (unsigned int i = 0; i < menuitems.size(); i++) {
+    for (size_t i = 0; i < menuitems.size(); i++) {
         clearItem(i, false);   // no clear
     }
 }
 
 void Menu::redrawFrame(FbDrawable &drawable) {
-    for (unsigned int i = 0; i < menuitems.size(); i++) {
+    for (size_t i = 0; i < menuitems.size(); i++) {
         drawItem(drawable, i);
     }
 
@@ -636,7 +636,8 @@ void Menu::redrawTitle(FbDrawable &drawable) {
     const char *text = menu.label.c_str();
 
     const FbTk::Font &font = theme().titleFont();
-    int dx = theme().bevelWidth(), len = menu.label.size();
+    int dx = theme().bevelWidth();
+    size_t len = menu.label.size();
     unsigned int l = font.textWidth(text, len) + theme().bevelWidth()*2;
 
     switch (theme().titleFontJustify()) {
diff --git a/src/FbTk/Menu.hh b/src/FbTk/Menu.hh
index bec76ac..fceec67 100644
--- a/src/FbTk/Menu.hh
+++ b/src/FbTk/Menu.hh
@@ -147,7 +147,7 @@ public:
     inline int y() const { return menu.window.y(); }
     inline unsigned int width() const { return menu.window.width(); }
     inline unsigned int height() const { return menu.window.height(); }
-    inline unsigned int numberOfItems() const { return menuitems.size(); }
+    inline size_t numberOfItems() const { return menuitems.size(); }
     inline int currentSubmenu() const { return m_which_sub; } 
     bool hasSubmenu(unsigned int index) const;
     bool isItemSelected(unsigned int index) const;
diff --git a/src/FbTk/MenuItem.cc b/src/FbTk/MenuItem.cc
index 93adc44..f39d03c 100644
--- a/src/FbTk/MenuItem.cc
+++ b/src/FbTk/MenuItem.cc
@@ -168,8 +168,8 @@ void MenuItem::draw(FbDrawable &draw,
             // disable clip mask
             XSetClipMask(disp, gc, None);
         } else if (isSelected()) {
-                draw.fillRectangle(theme.hiliteGC().gc(),
-                                   sel_x+item_pm_height/4, sel_y+item_pm_height/4, item_pm_height/2, item_pm_height/2);
+            draw.fillRectangle(theme.hiliteGC().gc(),
+                               sel_x+item_pm_height/4, sel_y+item_pm_height/4, item_pm_height/2, item_pm_height/2);
         }
     }
 
diff --git a/src/FbTk/MultLayers.cc b/src/FbTk/MultLayers.cc
index 6a6dc18..439201a 100644
--- a/src/FbTk/MultLayers.cc
+++ b/src/FbTk/MultLayers.cc
@@ -180,8 +180,8 @@ void MultLayers::restack() {
 }
 
 int MultLayers::size() {
-    int i = 0, num = 0;
-    for (; i < static_cast<signed>(m_layers.size()); i++) {
+    int num = 0;
+    for (size_t i = 0; i < m_layers.size(); i++) {
         num += m_layers[i]->countWindows();
     }
     return num;
diff --git a/src/FbTk/StringUtil.cc b/src/FbTk/StringUtil.cc
index d8b64c9..1a6814c 100644
--- a/src/FbTk/StringUtil.cc
+++ b/src/FbTk/StringUtil.cc
@@ -123,7 +123,7 @@ string findExtension(const string &filename) {
 string replaceString(const string &original,
                      const char *findthis,
                      const char *replace) {
-    int i=0;
+    size_t i = 0;
     const int size_of_replace = strlen(replace);
     const int size_of_find = strlen(findthis);
     string ret_str(original);
diff --git a/src/FbTk/TextButton.cc b/src/FbTk/TextButton.cc
index 1d5936c..ad9de09 100644
--- a/src/FbTk/TextButton.cc
+++ b/src/FbTk/TextButton.cc
@@ -144,7 +144,7 @@ void TextButton::renderForeground(FbWindow &win, FbDrawable &drawable) {
 }
 
 void TextButton::drawText(int x_offset, int y_offset, FbDrawable *drawable) {
-    unsigned int textlen = text().size();
+    size_t textlen = text().size();
     // do text alignment
 
     unsigned int textw = width(), texth = height();