From c8bcee597b7e4198b2d68682fa67e44413a4eee7 Mon Sep 17 00:00:00 2001 From: Fabio D'Urso Date: Mon, 20 May 2013 19:57:35 +0200 Subject: [PATCH] WidgetAnnotTools: Tidy up slots --- conf/widgetannottools.cpp | 23 +++++++++-------------- conf/widgetannottools.h | 12 +++++------- 2 files changed, 14 insertions(+), 21 deletions(-) diff --git a/conf/widgetannottools.cpp b/conf/widgetannottools.cpp index 1a8566436..790ee508e 100644 --- a/conf/widgetannottools.cpp +++ b/conf/widgetannottools.cpp @@ -64,11 +64,11 @@ WidgetAnnotTools::WidgetAnnotTools( QWidget * parent ) hBoxLayout->addLayout( vBoxLayout ); connect( m_list, SIGNAL( itemActivated(QListWidgetItem*) ), this, SLOT( slotItemActivated(QListWidgetItem*) ) ); - connect( m_list, SIGNAL( currentRowChanged(int) ), this, SLOT( slotRowChanged(int) ) ); - connect( m_btnAdd, SIGNAL( clicked(bool) ), this, SLOT( slotAdd(bool) ) ); - connect( m_btnRemove, SIGNAL( clicked(bool) ), this, SLOT( slotRemove(bool) ) ); - connect( m_btnMoveUp, SIGNAL( clicked(bool) ), this, SLOT( slotMoveUp(bool) ) ); - connect( m_btnMoveDown, SIGNAL( clicked(bool) ), this, SLOT( slotMoveDown(bool) ) ); + connect( m_list, SIGNAL( currentRowChanged(int) ), this, SLOT( updateButtons() ) ); + connect( m_btnAdd, SIGNAL( clicked(bool) ), this, SLOT( slotAdd() ) ); + connect( m_btnRemove, SIGNAL( clicked(bool) ), this, SLOT( slotRemove() ) ); + connect( m_btnMoveUp, SIGNAL( clicked(bool) ), this, SLOT( slotMoveUp() ) ); + connect( m_btnMoveDown, SIGNAL( clicked(bool) ), this, SLOT( slotMoveDown() ) ); } WidgetAnnotTools::~WidgetAnnotTools() @@ -189,12 +189,7 @@ void WidgetAnnotTools::slotItemActivated( QListWidgetItem *listEntry ) emit changed(); } -void WidgetAnnotTools::slotRowChanged( int ) -{ - updateButtons(); -} - -void WidgetAnnotTools::slotAdd( bool ) +void WidgetAnnotTools::slotAdd() { EditAnnotToolDialog t( this ); @@ -224,7 +219,7 @@ void WidgetAnnotTools::slotAdd( bool ) emit changed(); } -void WidgetAnnotTools::slotRemove( bool ) +void WidgetAnnotTools::slotRemove() { const int row = m_list->currentRow(); delete m_list->takeItem(row); @@ -232,7 +227,7 @@ void WidgetAnnotTools::slotRemove( bool ) emit changed(); } -void WidgetAnnotTools::slotMoveUp( bool ) +void WidgetAnnotTools::slotMoveUp() { const int row = m_list->currentRow(); m_list->insertItem( row, m_list->takeItem(row-1) ); @@ -241,7 +236,7 @@ void WidgetAnnotTools::slotMoveUp( bool ) emit changed(); } -void WidgetAnnotTools::slotMoveDown( bool ) +void WidgetAnnotTools::slotMoveDown() { const int row = m_list->currentRow(); m_list->insertItem( row, m_list->takeItem(row+1) ); diff --git a/conf/widgetannottools.h b/conf/widgetannottools.h index 1f851908d..df0db3e1f 100644 --- a/conf/widgetannottools.h +++ b/conf/widgetannottools.h @@ -45,8 +45,6 @@ class WidgetAnnotTools : public QWidget void changed(); private: - void updateButtons(); - QListWidget *m_list; KPushButton *m_btnAdd; KPushButton *m_btnRemove; @@ -55,11 +53,11 @@ class WidgetAnnotTools : public QWidget private slots: void slotItemActivated( QListWidgetItem * ); - void slotRowChanged( int ); - void slotAdd( bool ); - void slotRemove( bool ); - void slotMoveUp( bool ); - void slotMoveDown( bool ); + void updateButtons(); + void slotAdd(); + void slotRemove(); + void slotMoveUp(); + void slotMoveDown(); }; class EditAnnotToolDialog : public KDialog