Cleanup SplashScreen class

Cleaning up after replacing the QSplashScreen base class with the
QWidget class.
This commit is contained in:
Hennadii Stepanov 2018-12-02 01:26:28 +02:00
parent ed12fd83ca
commit 7d1b60ce93
No known key found for this signature in database
GPG key ID: 410108112E7EA81F
3 changed files with 9 additions and 12 deletions

View file

@ -217,7 +217,7 @@ Q_SIGNALS:
void requestedInitialize(); void requestedInitialize();
void requestedShutdown(); void requestedShutdown();
void stopThread(); void stopThread();
void splashFinished(QWidget *window); void splashFinished();
private: private:
QThread *coreThread; QThread *coreThread;
@ -357,9 +357,9 @@ void BitcoinApplication::createSplashScreen(const NetworkStyle *networkStyle)
{ {
SplashScreen *splash = new SplashScreen(m_node, 0, networkStyle); SplashScreen *splash = new SplashScreen(m_node, 0, networkStyle);
// We don't hold a direct pointer to the splash screen after creation, but the splash // We don't hold a direct pointer to the splash screen after creation, but the splash
// screen will take care of deleting itself when slotFinish happens. // screen will take care of deleting itself when finish() happens.
splash->show(); splash->show();
connect(this, &BitcoinApplication::splashFinished, splash, &SplashScreen::slotFinish); connect(this, &BitcoinApplication::splashFinished, splash, &SplashScreen::finish);
connect(this, &BitcoinApplication::requestedShutdown, splash, &QWidget::close); connect(this, &BitcoinApplication::requestedShutdown, splash, &QWidget::close);
} }
@ -500,7 +500,7 @@ void BitcoinApplication::initializeResult(bool success)
{ {
window->show(); window->show();
} }
Q_EMIT splashFinished(window); Q_EMIT splashFinished();
#ifdef ENABLE_WALLET #ifdef ENABLE_WALLET
// Now that initialization/startup is done, process any command-line // Now that initialization/startup is done, process any command-line
@ -514,7 +514,7 @@ void BitcoinApplication::initializeResult(bool success)
#endif #endif
pollShutdownTimer->start(200); pollShutdownTimer->start(200);
} else { } else {
Q_EMIT splashFinished(window); // Make sure splash screen doesn't stick around during shutdown Q_EMIT splashFinished(); // Make sure splash screen doesn't stick around during shutdown
quit(); // Exit first main loop invocation quit(); // Exit first main loop invocation
} }
} }

View file

@ -145,10 +145,8 @@ bool SplashScreen::eventFilter(QObject * obj, QEvent * ev) {
return QObject::eventFilter(obj, ev); return QObject::eventFilter(obj, ev);
} }
void SplashScreen::slotFinish(QWidget *mainWin) void SplashScreen::finish()
{ {
Q_UNUSED(mainWin);
/* If the window is minimized, hide() will be ignored. */ /* If the window is minimized, hide() will be ignored. */
/* Make sure we de-minimize the splashscreen window before hiding */ /* Make sure we de-minimize the splashscreen window before hiding */
if (isMinimized()) if (isMinimized())

View file

@ -5,8 +5,7 @@
#ifndef BITCOIN_QT_SPLASHSCREEN_H #ifndef BITCOIN_QT_SPLASHSCREEN_H
#define BITCOIN_QT_SPLASHSCREEN_H #define BITCOIN_QT_SPLASHSCREEN_H
#include <functional> #include <QWidget>
#include <QSplashScreen>
#include <memory> #include <memory>
@ -37,8 +36,8 @@ protected:
void closeEvent(QCloseEvent *event); void closeEvent(QCloseEvent *event);
public Q_SLOTS: public Q_SLOTS:
/** Slot to call finish() method as it's not defined as slot */ /** Hide the splash screen window and schedule the splash screen object for deletion */
void slotFinish(QWidget *mainWin); void finish();
/** Show message and progress */ /** Show message and progress */
void showMessage(const QString &message, int alignment, const QColor &color); void showMessage(const QString &message, int alignment, const QColor &color);