From c5e27b283b19e60bb219b01b130997783188905d Mon Sep 17 00:00:00 2001 From: heinrich5991 Date: Tue, 1 Aug 2023 09:45:15 +0200 Subject: [PATCH] Delete fishy-looking assignment operator/copy constructor of `IJob` Since they're not called anyway, they can go away. --- src/engine/shared/jobs.cpp | 11 ----------- src/engine/shared/jobs.h | 4 ++-- 2 files changed, 2 insertions(+), 13 deletions(-) diff --git a/src/engine/shared/jobs.cpp b/src/engine/shared/jobs.cpp index 7141d3f07..be21482ce 100644 --- a/src/engine/shared/jobs.cpp +++ b/src/engine/shared/jobs.cpp @@ -9,17 +9,6 @@ IJob::IJob() : { } -IJob::IJob(const IJob &Other) : - m_Status(STATE_PENDING) -{ -} - -IJob &IJob::operator=(const IJob &Other) -{ - m_Status = STATE_PENDING; - return *this; -} - IJob::~IJob() = default; int IJob::Status() diff --git a/src/engine/shared/jobs.h b/src/engine/shared/jobs.h index 642299efc..fa99166f9 100644 --- a/src/engine/shared/jobs.h +++ b/src/engine/shared/jobs.h @@ -22,8 +22,8 @@ private: public: IJob(); - IJob(const IJob &Other); - IJob &operator=(const IJob &Other); + IJob(const IJob &Other) = delete; + IJob &operator=(const IJob &Other) = delete; virtual ~IJob(); int Status();