[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdelibs/kdeui (silent)
From:       Ravikiran Rajagopal <ravi () kde ! org>
Date:       2003-05-27 17:12:56
[Download RAW message or body]

CVS commit by ravi: 

CVS_SILENT Remove obsolete comments, and a couple of whitespace fixes.


  M +5 -16     kjanuswidget.cpp   1.63


--- kdelibs/kdeui/kjanuswidget.cpp  #1.62:1.63
@@ -1,4 +1,5 @@
 /*  This file is part of the KDE Libraries
  *  Copyright (C) 1999-2000 Espen Sand (espensa@online.no)
+ *  Copyright (C) 2003 Ravikiran Rajagopal (ravi@kde.org)
  *
  *  This library is free software; you can redistribute it and/or
@@ -61,7 +62,4 @@ class KJanusWidget::IconListItem : publi
 };
 
-// Added to make sure that the indices returned by KJanusWidget::pageIndex()
-// remain constant and safe to store.
-// Ravi <ravi@ee.eng.ohio-state.edu> Sun Feb 23 2003
 class KJanusWidget::KJanusWidgetPrivate
 {
@@ -443,11 +441,4 @@ void KJanusWidget::addPageWidget( QFrame
       QString itemName = items.last();
       IconListItem *item = new IconListItem( mIconList, pixmap, itemName );
-      //
-      // 2000-06-01 Espen Sand: If I do this with Qt 2.1.1 all sorts of
-      // strange things happen. With Qt <= 2.1 it worked but now I must
-      // either specify the listbox in the constructor on the item
-      // or as below, not both.
-      // mIconList->insertItem( item );
-      //
       mIconListToPageStack.insert(item, page);
       mIconList->invalidateHeight();
@@ -1055,8 +1046,6 @@ void KJanusWidget::virtual_hook( int, vo
 { /*BASE::virtual_hook( id, data );*/ }
 
-// Just remove the page from our stack of widgets. Do not modify the given widget in
-// any way. No memory leak occurs as parent is not changed.
-// Make this virtual in KDE 4.0.
-// Ravikiran Rajagopal <ravi@ee.eng.ohio-state.edu>
+// TODO: In TreeList, if the last child of a node is removed, and there is no \
corrsponding widget for that node, allow the caller to +// delete the node.
 void KJanusWidget::removePage( QWidget *page )
 {


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic