style fixes, i suck

svn path=/trunk/KDE/kdegraphics/okular/; revision=846875

ui/side_reviews.cpp

@@ -212,24 +212,20 @@ void Reviews::activated( const QModelIndex &index )
     m_document->setViewport( vp, -1, true );
 }
 
-QModelIndexList Reviews::retrieveAnnotations(const QModelIndex& idx)
+QList<QModelIndex> Reviews::retrieveAnnotations(const QModelIndex& idx) const
 {
-    QModelIndexList ret;
+    QList<QModelIndex> ret;
-    if ( idx.isValid() )
+    if(idx.isValid() && idx.model()->hasChildren(idx))
     {
-        if ( idx.model()->hasChildren( idx ) )
+        for(int i=0; i<idx.model()->rowCount(idx); i++)
-        {
-            int rowCount = idx.model()->rowCount( idx );
-            for ( int i = 0; i < rowCount; i++ )
         {
             ret += retrieveAnnotations(idx.child(i, idx.column()));
         }
     }
-        else
+    else if(idx.isValid())
     {
         ret += idx;
     }
-    }
     
     return ret;
 }
@@ -245,7 +241,7 @@ void Reviews::contextMenuRequested( const QPoint &pos )
     QModelIndexList indexes = m_view->selectionModel()->selectedIndexes();
     Q_FOREACH ( const QModelIndex &index, indexes )
     {
-        QModelIndexList annotations = retrieveAnnotations(index);
+        QList<QModelIndex> annotations = retrieveAnnotations(index);
         Q_FOREACH(const QModelIndex &idx, annotations)
         {
             const QModelIndex authorIndex = m_authorProxy->mapToSource( idx );

ui/side_reviews.h

@@ -56,7 +56,7 @@ class Reviews : public QWidget, public Okular::DocumentObserver
         void contextMenuRequested( const QPoint& );
 
     private:
-        QModelIndexList retrieveAnnotations(const QModelIndex& idx);
+        QModelIndexList retrieveAnnotations(const QModelIndex& idx) const;
         
         // data fields (GUI)
         TreeView * m_view;