diff options
author | Tobias Markmann <tm@ayena.de> | 2015-11-03 14:56:45 (GMT) |
---|---|---|
committer | Kevin Smith <kevin.smith@isode.com> | 2015-11-06 10:53:47 (GMT) |
commit | d3d767ba12d88fcb8d16b9a70f95781bacb19bfc (patch) | |
tree | 1831b894f43dfccd46458297f8f33bf66b715a4f /SwifTools/Application | |
parent | 7302ee18a137f6810ef1cc40a395f99d2f46a644 (diff) | |
download | swift-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/Application')
-rw-r--r-- | SwifTools/Application/ApplicationPathProvider.h | 9 |
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; |