deleteLater is unecessary in this case and it just screws up some of our use cases such as the screen saver toolbox
svn path=/trunk/KDE/kdelibs/; revision=885859
This commit is contained in:
parent
649057791d
commit
1171ef67d8
@ -119,6 +119,7 @@ void ToolBox::addTool(QAction *action)
|
|||||||
|
|
||||||
//make enabled/disabled tools appear/disappear instantly
|
//make enabled/disabled tools appear/disappear instantly
|
||||||
connect(tool, SIGNAL(changed()), this, SLOT(updateToolBox()));
|
connect(tool, SIGNAL(changed()), this, SLOT(updateToolBox()));
|
||||||
|
kDebug() << "added tool" << action->text() << (QGraphicsItem*)tool;
|
||||||
}
|
}
|
||||||
|
|
||||||
void ToolBox::updateToolBox()
|
void ToolBox::updateToolBox()
|
||||||
@ -136,7 +137,7 @@ void ToolBox::removeTool(QAction *action)
|
|||||||
Plasma::IconWidget *tool = dynamic_cast<Plasma::IconWidget*>(child);
|
Plasma::IconWidget *tool = dynamic_cast<Plasma::IconWidget*>(child);
|
||||||
if (tool && tool->action() == action) {
|
if (tool && tool->action() == action) {
|
||||||
//kDebug() << "tool found!";
|
//kDebug() << "tool found!";
|
||||||
tool->deleteLater();
|
delete tool;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user