summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTobias Markmann <tm@ayena.de>2015-11-03 14:56:45 (GMT)
committerKevin Smith <kevin.smith@isode.com>2015-11-06 10:53:47 (GMT)
commitd3d767ba12d88fcb8d16b9a70f95781bacb19bfc (patch)
tree1831b894f43dfccd46458297f8f33bf66b715a4f /SwifTools
parent7302ee18a137f6810ef1cc40a395f99d2f46a644 (diff)
downloadswift-d3d767ba12d88fcb8d16b9a70f95781bacb19bfc.zip
swift-d3d767ba12d88fcb8d16b9a70f95781bacb19bfc.tar.bz2
Make methods non-virtual that are not overridden by subclass
These cases have been uncovered by Clang Analyzer as they were called during some constructor. However, there is no reason for them being virtual. They are never overridden in a derived class. Test-Information: Unit tests still pass on OS X 10.10.5. Change-Id: I90b5396a0dba971d9150c35f9107733f7f4637cd
Diffstat (limited to 'SwifTools')
-rw-r--r--SwifTools/Application/ApplicationPathProvider.h9
1 files changed, 5 insertions, 4 deletions
diff --git a/SwifTools/Application/ApplicationPathProvider.h b/SwifTools/Application/ApplicationPathProvider.h
index 17c39dd..aba9687 100644
--- a/SwifTools/Application/ApplicationPathProvider.h
+++ b/SwifTools/Application/ApplicationPathProvider.h
@@ -1,14 +1,15 @@
/*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2015 Isode Limited.
* All rights reserved.
* See the COPYING file for more information.
*/
#pragma once
-#include <boost/filesystem.hpp>
-#include <vector>
#include <string>
+#include <vector>
+
+#include <boost/filesystem.hpp>
namespace Swift {
class ApplicationPathProvider {
@@ -18,7 +19,7 @@ namespace Swift {
virtual boost::filesystem::path getHomeDir() const = 0;
virtual boost::filesystem::path getDataDir() const = 0;
- virtual boost::filesystem::path getExecutableDir() const;
+ boost::filesystem::path getExecutableDir() const;
boost::filesystem::path getProfileDir(const std::string& profile) const;
boost::filesystem::path getResourcePath(const std::string& resource) const;