[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