[Uml-devel] KDE/kdesdk/umbrello/umbrello

[Andi Fischer]

SVN commit 1269446 by fischer:

Some Krazy2 issues fixed.

 M  +0 -1      umllistview.cpp  
 M  +0 -1      umllistview.h  
 M  +6 -6      umllistviewitem.cpp  


--- trunk/KDE/kdesdk/umbrello/umbrello/umllistview.cpp #1269445:1269446
@@ -17,7 +17,6 @@
 #include <QtCore/QPoint>
 #include <QtCore/QRect>
 #include <QtCore/QEvent>
-#include <q3header.h>
 #include <QFocusEvent>
 #include <QKeyEvent>
 #include <QDropEvent>
--- trunk/KDE/kdesdk/umbrello/umbrello/umllistview.h #1269445:1269446
@@ -21,7 +21,6 @@
 #include <QtXml/QDomElement>
 #include <QtGui/QTreeWidget>
 #include <QtGui/QTreeWidgetItem>
-#include <QtGui/QTreeWidget>
 
 class QEvent;
 class QMouseEvent;
--- trunk/KDE/kdesdk/umbrello/umbrello/umllistviewitem.cpp #1269445:1269446
@@ -260,8 +260,7 @@
  * Sets the id this class represents.
  * This only sets the ID locally, not at the UMLObject that is perhaps
  * associated to this UMLListViewItem.
- *
- * @return  The id this class represents.
+ * @param id   the id this class represents
  */
 void UMLListViewItem::setID(Uml::IDType id)
 {
@@ -452,7 +451,6 @@
     }
 }
 
-
 /**
  * This function is called if the user presses Enter during in-place renaming
  * of the item in column col, reimplemented from QlistViewItem
@@ -875,14 +873,16 @@
  */
 UMLListViewItem * UMLListViewItem::findItem(Uml::IDType id)
 {
-    if (getID() == id)
+    if (getID() == id) {
         return this;
-    for (int i=0; i < childCount(); i++) {
+    }
+    for (int i = 0; i < childCount(); ++i) {
         UMLListViewItem *childItem = static_cast<UMLListViewItem*>(child(i));
         UMLListViewItem *inner = childItem->findItem(id);
-        if (inner)
+        if (inner) {
             return inner;
     }
+    }
     return NULL;
 }