fix some warning and a bogus empty test.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@21053 a592a061-630c-0410-9148-cb99ea01b6c8

diff --git a/src/frontends/qt4/GuiPrefs.cpp b/src/frontends/qt4/GuiPrefs.cpp
index f35a0f85bf03b2cb44ce7b867e072595217dc087..0c74b58857ca0b3ac4fb3b3331166c76279bcd1d 100644 (file)
--- a/src/frontends/qt4/GuiPrefs.cpp
+++ b/src/frontends/qt4/GuiPrefs.cpp
@@ -1852,7 +1852,7 @@ void PrefShortcuts::on_modifyPB_pressed()
 void PrefShortcuts::on_removePB_pressed()
 {
        QList<QTreeWidgetItem*> items = shortcutsTW->selectedItems();
-       for (size_t i = 0; i < items.size(); ++i)
+       for (int i = 0; i < items.size(); ++i)
                items[i]->setText(1, QString());
 }
 
@@ -1863,20 +1863,20 @@ void PrefShortcuts::on_searchPB_pressed()
                searchLE->text(), 
                Qt::MatchFlags(Qt::MatchContains | Qt::MatchRecursive));
        
-       if (!matched.size() > 0)
+       if (matched.isEmpty())
                return;
        
        QList<QTreeWidgetItem *> const items = shortcutsTW->selectedItems();
        // clear current selection
-       for (size_t i = 0; i < items.size(); ++i)
+       for (int i = 0; i < items.size(); ++i)
                items[i]->setSelected(false);
-       for (size_t i = 0; i < matched.size(); ++i) {
+       for (int i = 0; i < matched.size(); ++i) {
                if (matched[i]->flags() & Qt::ItemIsSelectable)
                        matched[i]->setSelected(true);
                matched[i]->setExpanded(true);
        }
        // scroll to the first selectable item
-       for (size_t i = 0; i < matched.size(); ++i)
+       for (int i = 0; i < matched.size(); ++i)
                if (matched[i]->flags() & Qt::ItemIsSelectable) {
                        shortcutsTW->scrollToItem(matched[i]);
                        break;