# HG changeset patch # User Hasan Yavuz Ă–ZDERYA # Date 2015-09-29 17:26:12 # Node ID 084c3dc65b46ca824e6c04823e3f5da2c5941894 # Parent dd74a99bcfac577847f4f529e7deaf93e9140cd3 some code cleanup, added `addSnapshot` function diff --git a/snapshotmanager.cpp b/snapshotmanager.cpp --- a/snapshotmanager.cpp +++ b/snapshotmanager.cpp @@ -57,12 +57,15 @@ void SnapshotManager::takeSnapshot() snapShot->data[ci][i] = _channelBuffers->at(ci)->sample(i); } } - snapshots.append(snapShot); - QObject::connect(snapShot, &SnapShot::deleteRequested, + + addSnapshot(snapShot); +} + +void SnapshotManager::addSnapshot(SnapShot* snapshot) +{ + snapshots.append(snapshot); + QObject::connect(snapshot, &SnapShot::deleteRequested, this, &SnapshotManager::deleteSnapshot); - QObject::connect(snapShot, &SnapShot::deleteRequested, - this, &SnapshotManager::updateMenu); - updateMenu(); } @@ -152,16 +155,10 @@ void SnapshotManager::loadSnapshot() lineNum++; } - auto snapShot = new SnapShot(_mainWindow, QFileInfo(fileName).baseName()); - snapShot->data = data; + auto snapshot = new SnapShot(_mainWindow, QFileInfo(fileName).baseName()); + snapshot->data = data; - snapshots.append(snapShot); - QObject::connect(snapShot, &SnapShot::deleteRequested, - this, &SnapshotManager::deleteSnapshot); - QObject::connect(snapShot, &SnapShot::deleteRequested, - this, &SnapshotManager::updateMenu); - - updateMenu(); + addSnapshot(snapshot); } QMenu* SnapshotManager::menu() diff --git a/snapshotmanager.h b/snapshotmanager.h --- a/snapshotmanager.h +++ b/snapshotmanager.h @@ -33,6 +33,7 @@ private: QAction loadSnapshotAction; QAction clearAction; + void addSnapshot(SnapShot* snapshot); void updateMenu(); private slots: