From 74a50831631ca8e50876d3f4bc698ebf46a3ee6b Mon Sep 17 00:00:00 2001 From: Ivan Donchevskii Date: Thu, 7 Dec 2017 13:01:04 +0100 Subject: Utils: remove unused variable in SaveFile class Backup member was never set to true. Change-Id: I21dc889813820e2a463c5ba812b18b57766e03c4 Reviewed-by: Oswald Buddenhagen --- src/libs/utils/savefile.cpp | 5 ++--- src/libs/utils/savefile.h | 3 --- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/src/libs/utils/savefile.cpp b/src/libs/utils/savefile.cpp index aeac15a627..3e00b36bee 100644 --- a/src/libs/utils/savefile.cpp +++ b/src/libs/utils/savefile.cpp @@ -39,7 +39,7 @@ namespace Utils { QFile::Permissions SaveFile::m_umask = 0; SaveFile::SaveFile(const QString &filename) : - m_finalFileName(filename), m_finalized(true), m_backup(false) + m_finalFileName(filename), m_finalized(true) { } @@ -128,8 +128,7 @@ bool SaveFile::commit() remove(); return false; } - if (!m_backup) - QFile::remove(bakname); + QFile::remove(bakname); return true; } diff --git a/src/libs/utils/savefile.h b/src/libs/utils/savefile.h index 3313574a39..c52baf6957 100644 --- a/src/libs/utils/savefile.h +++ b/src/libs/utils/savefile.h @@ -44,14 +44,11 @@ public: void rollback(); bool commit(); - void setBackup(bool backup) { m_backup = backup; } - static void initializeUmask(); private: const QString m_finalFileName; bool m_finalized; - bool m_backup; static QFile::Permissions m_umask; }; -- cgit v1.2.1