Merge #17252: 0.19: gui: Make polling in ClientModel asynchronous
d5c36ce0c4
gui: Make polling in ClientModel asynchronous (João Barbosa) Pull request description: Backport #17135. ACKs for top commit: laanwj: ACKd5c36ce0c4
, it is a clean cherry-pick of 6b6be41c36e4fe9a74bed50e7f0a06532ab1260b. Tree-SHA512: 4e514f205866d87bdc19a57dede2214891237d7b663c9c8c9f19a9ab5c5a6e64876065bebb6c16a1799b02e0eb971318866b4e0824155b47063ce379fb0155e2
This commit is contained in:
commit
0f6f7a574a
2 changed files with 21 additions and 14 deletions
|
@ -19,6 +19,7 @@
|
|||
#include <stdint.h>
|
||||
|
||||
#include <QDebug>
|
||||
#include <QThread>
|
||||
#include <QTimer>
|
||||
|
||||
static int64_t nLastHeaderTipUpdateNotification = 0;
|
||||
|
@ -30,15 +31,26 @@ ClientModel::ClientModel(interfaces::Node& node, OptionsModel *_optionsModel, QO
|
|||
optionsModel(_optionsModel),
|
||||
peerTableModel(nullptr),
|
||||
banTableModel(nullptr),
|
||||
pollTimer(nullptr)
|
||||
m_thread(new QThread(this))
|
||||
{
|
||||
cachedBestHeaderHeight = -1;
|
||||
cachedBestHeaderTime = -1;
|
||||
peerTableModel = new PeerTableModel(m_node, this);
|
||||
banTableModel = new BanTableModel(m_node, this);
|
||||
pollTimer = new QTimer(this);
|
||||
connect(pollTimer, &QTimer::timeout, this, &ClientModel::updateTimer);
|
||||
pollTimer->start(MODEL_UPDATE_DELAY);
|
||||
|
||||
QTimer* timer = new QTimer;
|
||||
timer->setInterval(MODEL_UPDATE_DELAY);
|
||||
connect(timer, &QTimer::timeout, [this] {
|
||||
// no locking required at this point
|
||||
// the following calls will acquire the required lock
|
||||
Q_EMIT mempoolSizeChanged(m_node.getMempoolSize(), m_node.getMempoolDynamicUsage());
|
||||
Q_EMIT bytesChanged(m_node.getTotalBytesRecv(), m_node.getTotalBytesSent());
|
||||
});
|
||||
connect(m_thread, &QThread::finished, timer, &QObject::deleteLater);
|
||||
connect(m_thread, &QThread::started, [timer] { timer->start(); });
|
||||
// move timer to thread so that polling doesn't disturb main event loop
|
||||
timer->moveToThread(m_thread);
|
||||
m_thread->start();
|
||||
|
||||
subscribeToCoreSignals();
|
||||
}
|
||||
|
@ -46,6 +58,9 @@ ClientModel::ClientModel(interfaces::Node& node, OptionsModel *_optionsModel, QO
|
|||
ClientModel::~ClientModel()
|
||||
{
|
||||
unsubscribeFromCoreSignals();
|
||||
|
||||
m_thread->quit();
|
||||
m_thread->wait();
|
||||
}
|
||||
|
||||
int ClientModel::getNumConnections(unsigned int flags) const
|
||||
|
@ -90,14 +105,6 @@ int64_t ClientModel::getHeaderTipTime() const
|
|||
return cachedBestHeaderTime;
|
||||
}
|
||||
|
||||
void ClientModel::updateTimer()
|
||||
{
|
||||
// no locking required at this point
|
||||
// the following calls will acquire the required lock
|
||||
Q_EMIT mempoolSizeChanged(m_node.getMempoolSize(), m_node.getMempoolDynamicUsage());
|
||||
Q_EMIT bytesChanged(m_node.getTotalBytesRecv(), m_node.getTotalBytesSent());
|
||||
}
|
||||
|
||||
void ClientModel::updateNumConnections(int numConnections)
|
||||
{
|
||||
Q_EMIT numConnectionsChanged(numConnections);
|
||||
|
|
|
@ -90,7 +90,8 @@ private:
|
|||
PeerTableModel *peerTableModel;
|
||||
BanTableModel *banTableModel;
|
||||
|
||||
QTimer *pollTimer;
|
||||
//! A thread to interact with m_node asynchronously
|
||||
QThread* const m_thread;
|
||||
|
||||
void subscribeToCoreSignals();
|
||||
void unsubscribeFromCoreSignals();
|
||||
|
@ -110,7 +111,6 @@ Q_SIGNALS:
|
|||
void showProgress(const QString &title, int nProgress);
|
||||
|
||||
public Q_SLOTS:
|
||||
void updateTimer();
|
||||
void updateNumConnections(int numConnections);
|
||||
void updateNetworkActive(bool networkActive);
|
||||
void updateAlert();
|
||||
|
|
Loading…
Add table
Reference in a new issue