summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorAlbert Astals Cid <aacid@kde.org>2018-01-08 18:07:38 +0100
committerAlbert Astals Cid <aacid@kde.org>2018-01-08 18:37:33 +0100
commit8794789a72f845b009656e6d7ae6a00b709e09bc (patch)
tree82220f5640d4abcf726f6e83e59cd6bb59cc4d67 /test
parent4c7814342806b61fedbb2d45ce74462f9dbc20bc (diff)
Delete lots of copy constructors and copy assignment operators
Fixes rule-of-three and copyable-polymorphic warnings reported by clazy. The default copy constructor and copy assignment operator are only valid for simple classes so we delete them (i.e. make then not exist) when we have either a virtual class or a destructor, the code still compiles so this doesn't fix any bug, it is more a protection for when you think you can copy a class and don't realize the default copy constrcutor is not doing what you want and you get crashes. Hopefully this helps us in the future :)
Diffstat (limited to 'test')
-rw-r--r--test/perf-test.cc4
1 files changed, 4 insertions, 0 deletions
diff --git a/test/perf-test.cc b/test/perf-test.cc
index 78083275..7710655e 100644
--- a/test/perf-test.cc
+++ b/test/perf-test.cc
@@ -1,5 +1,6 @@
/* Copyright Krzysztof Kowalczyk 2006-2007
Copyright Hib Eris <hib@hiberis.nl> 2008, 2013
+ Copyright 2018 Albert Astals Cid <aacid@kde.org> 2018
License: GPLv2 */
/*
A tool to stress-test poppler rendering and measure rendering times for
@@ -82,6 +83,9 @@ public:
PdfEnginePoppler();
~PdfEnginePoppler();
+ PdfEnginePoppler(const PdfEnginePoppler &) = delete;
+ PdfEnginePoppler& operator=(const PdfEnginePoppler &) = delete;
+
const char *fileName(void) const { return _fileName; };
void setFileName(const char *fileName) {