Browse Source

Use atomic primitives from standard library

QAtomicInteger underlying is using std::atomic structures, so
using std::atomic directly should not be a problem for us.

PR #19507.
adaptive-webui-19844
Chocobo1 1 year ago committed by GitHub
parent
commit
d8a03cd8d8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      src/app/application.cpp
  2. 5
      src/app/application.h
  3. 4
      src/base/bittorrent/torrentcreator.cpp
  4. 5
      src/base/bittorrent/torrentcreator.h

2
src/app/application.cpp

@ -1274,7 +1274,7 @@ void Application::adjustThreadPriority() const
void Application::cleanup() void Application::cleanup()
{ {
// cleanup() can be called multiple times during shutdown. We only need it once. // cleanup() can be called multiple times during shutdown. We only need it once.
if (!m_isCleanupRun.testAndSetAcquire(0, 1)) if (m_isCleanupRun.exchange(true, std::memory_order_acquire))
return; return;
LogMsg(tr("qBittorrent termination initiated")); LogMsg(tr("qBittorrent termination initiated"));

5
src/app/application.h

@ -30,8 +30,9 @@
#pragma once #pragma once
#include <atomic>
#include <QtSystemDetection> #include <QtSystemDetection>
#include <QAtomicInt>
#include <QCoreApplication> #include <QCoreApplication>
#include <QPointer> #include <QPointer>
#include <QStringList> #include <QStringList>
@ -177,7 +178,7 @@ private:
#endif #endif
ApplicationInstanceManager *m_instanceManager = nullptr; ApplicationInstanceManager *m_instanceManager = nullptr;
QAtomicInt m_isCleanupRun; std::atomic_bool m_isCleanupRun;
bool m_isProcessingParamsAllowed = false; bool m_isProcessingParamsAllowed = false;
ShutdownDialogAction m_shutdownAct = ShutdownDialogAction::Exit; ShutdownDialogAction m_shutdownAct = ShutdownDialogAction::Exit;
QBtCommandLineParameters m_commandLineArgs; QBtCommandLineParameters m_commandLineArgs;

4
src/base/bittorrent/torrentcreator.cpp

@ -93,12 +93,12 @@ void TorrentCreator::checkInterruptionRequested() const
void TorrentCreator::requestInterruption() void TorrentCreator::requestInterruption()
{ {
m_interruptionRequested.storeRelaxed(1); m_interruptionRequested.store(true, std::memory_order_relaxed);
} }
bool TorrentCreator::isInterruptionRequested() const bool TorrentCreator::isInterruptionRequested() const
{ {
return m_interruptionRequested.loadRelaxed() != 0; return m_interruptionRequested.load(std::memory_order_relaxed);
} }
void TorrentCreator::run() void TorrentCreator::run()

5
src/base/bittorrent/torrentcreator.h

@ -28,7 +28,8 @@
#pragma once #pragma once
#include <QAtomicInt> #include <atomic>
#include <QObject> #include <QObject>
#include <QRunnable> #include <QRunnable>
#include <QStringList> #include <QStringList>
@ -96,6 +97,6 @@ namespace BitTorrent
void checkInterruptionRequested() const; void checkInterruptionRequested() const;
TorrentCreatorParams m_params; TorrentCreatorParams m_params;
QAtomicInt m_interruptionRequested; std::atomic_bool m_interruptionRequested;
}; };
} }

Loading…
Cancel
Save