Missing Destructors: Issue #307 (#308)

* Adding empty destructor for Scene.

* RenderEngine Destructor empty/no longer default

* Create empty ~ScreenLog
This commit is contained in:
Matthew Territo
2017-05-18 02:13:20 -06:00
committed by Emil Axelsson
parent 4671928754
commit 0a1101d0e4
6 changed files with 19 additions and 3 deletions
+5
View File
@@ -181,6 +181,11 @@ RenderEngine::RenderEngine()
addProperty(_disableMasterRendering);
}
/**
* Destructor
*/
RenderEngine::~RenderEngine() {}
void RenderEngine::setRendererFromString(const std::string& renderingMethod) {
_rendererImplementation = rendererFromString(renderingMethod);
+3
View File
@@ -92,6 +92,9 @@ Scene::Scene()
)
{}
Scene::~Scene(){
}
void Scene::setRoot(std::unique_ptr<SceneGraphNode> root) {
if (_root) {
removeNode(_root.get());
+3 -1
View File
@@ -34,7 +34,9 @@ ScreenLog::ScreenLog(std::chrono::seconds timeToLive, LogLevel logLevel)
: _timeToLive(std::move(timeToLive))
, _logLevel(logLevel)
{}
ScreenLog::~ScreenLog() {}
void ScreenLog::removeExpiredEntries() {
auto t = std::chrono::steady_clock::now();
auto ttl = _timeToLive;