old/clippy-terminal
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Fix not virtual destructor

Browse source
This commit is contained in:
Timofey Khoruzhii 2022-10-28 23:49:52 +03:00
parent 80ca6ad02d
commit dd95598c2a
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/clippy/target.hpp
View file

@ -20,11 +20,13 @@ namespace clippy::targets {
class Target { class Target {
public: public:
virtual void Execute() = 0; virtual void Execute() = 0;
virtual ~Target() = 0;
}; };
class EmptyTarget : public Target { class EmptyTarget : public Target {
public: public:
void Execute() override {} void Execute() override {}
~EmptyTarget() override {}
}; };
class OpenProjectConfig : public Target { class OpenProjectConfig : public Target {
@ -32,6 +34,7 @@ class OpenProjectConfig : public Target {
OpenProjectConfig(Config config) : config_(config) {} OpenProjectConfig(Config config) : config_(config) {}
void Execute() override { config_.Edit(); } void Execute() override { config_.Edit(); }
~OpenProjectConfig() override {}
private: private:
Config config_; Config config_;
@ -48,6 +51,7 @@ class CreateProjectConfig : public Target {
auto config = projects_.GetNewConfig(scripts_path); auto config = projects_.GetNewConfig(scripts_path);
config.Edit(); config.Edit();
} }
~CreateProjectConfig() override {}
private: private:
ProjectList& projects_; ProjectList& projects_;
@ -85,6 +89,8 @@ class RunShellScript : public Target {
} }
} }
~RunShellScript() override {}
private: private:
std::vector<std::string> commands_; std::vector<std::string> commands_;
}; };