Code refactoring

[vlp.git] / src / lgconfig / NodeListDialog.cpp

diff --git a/src/lgconfig/NodeListDialog.cpp b/src/lgconfig/NodeListDialog.cpp
index 44567b088931f874731c1d0429ffa07684a42643..fb08ffec53e6c0d97b35066acd53cc527b3f431d 100644 (file)
--- a/src/lgconfig/NodeListDialog.cpp
+++ b/src/lgconfig/NodeListDialog.cpp
@@ -9,7 +9,7 @@ namespace loglan {
 namespace vlp {
 namespace dialog {
 
-NodeListDialog::NodeListDialog(QList<VLPEntry*> *nodes, QWidget * parent)
+NodeListDialog::NodeListDialog(QList<VLPEntry*> nodes, QWidget * parent)
        : QDialog(parent)
 {
        setupUi(this);
@@ -26,7 +26,7 @@ NodeListDialog::~NodeListDialog()
 
 bool NodeListDialog::check_id(int id)
 {
-       QListIterator<VLPEntry *> nodesIterator(*_nodes);
+       QListIterator<VLPEntry *> nodesIterator(_nodes);
        VLPEntry *node = NULL;
 
        while (nodesIterator.hasNext()) {
@@ -39,7 +39,7 @@ bool NodeListDialog::check_id(int id)
 
 bool NodeListDialog::check_addr(QString addr)
 {
-       QListIterator<VLPEntry *> nodesIterator(*_nodes);
+       QListIterator<VLPEntry *> nodesIterator(_nodes);
        VLPEntry *node = NULL;
 
        while (nodesIterator.hasNext()) {
@@ -50,6 +50,11 @@ bool NodeListDialog::check_addr(QString addr)
        return true;
 }
 
+QList<VLPEntry*> NodeListDialog::getNodes()
+{
+       return _nodes;
+}
+
 void NodeListDialog::on_addButton_clicked()
 {
        dialog::AddNodeDialog dialog(this);
@@ -81,7 +86,7 @@ void NodeListDialog::on_addButton_clicked()
                                QListWidgetItem * listWidgetItem = new QListWidgetItem;
                                listWidgetItem->setText(info);
                                nodeList->addItem(listWidgetItem);
-                               _nodes->append(vlpEntry);
+                               _nodes.append(vlpEntry);
                        }
                        else {
                                QMessageBox::warning(this,
@@ -103,7 +108,7 @@ void NodeListDialog::on_addButton_clicked()
 
 void NodeListDialog::removeNodes(QList<QListWidgetItem *> selectedNodes)
 {
-       QListIterator<VLPEntry *> nodesIterator(*_nodes);
+       QListIterator<VLPEntry *> nodesIterator(_nodes);
 
        fprintf(stderr, " > Removing nodes\n");
        for (auto selectedItem : selectedNodes) {
@@ -122,7 +127,7 @@ void NodeListDialog::removeNodes(QList<QListWidgetItem *> selectedNodes)
                        fprintf(stderr, "   - removed\n");
                        int row = nodeList->row(selectedItem);
                        delete nodeList->takeItem(row);
-                       _nodes->removeOne(node);
+                       _nodes.removeOne(node);
                }
        }
 }