From 47dff9395143460a22273f4f8f20204123d237c4 Mon Sep 17 00:00:00 2001 From: Thilo Graf Date: Thu, 2 Jun 2016 20:13:38 +0200 Subject: [PATCH] CMovieBrowser: remove hide() from destructor hide() in constructor could make problems on created moviebrowser instances if gui functionality of moviebrowser is not required, eg if only members are used without gui. Gui parts could be unintended removed from screen. Note: hide() must be called explicit now. I didn't found any problems at the moment, but please report if you found problems, so we can fix it. Origin commit data ------------------ Branch: ni/coolstream Commit: https://github.com/neutrino-images/ni-neutrino/commit/137598d36834eb0ceb98f757a7b7c9e3896c4938 Author: Thilo Graf Date: 2016-06-02 (Thu, 02 Jun 2016) ------------------ This commit was generated by Migit --- src/gui/moviebrowser.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gui/moviebrowser.cpp b/src/gui/moviebrowser.cpp index 71729f51e..ae4c5b550 100644 --- a/src/gui/moviebrowser.cpp +++ b/src/gui/moviebrowser.cpp @@ -351,7 +351,7 @@ CMovieBrowser::CMovieBrowser(): configfile ('\t') CMovieBrowser::~CMovieBrowser() { //TRACE("[mb] del\n"); - hide(); + m_dir.clear(); m_dirNames.clear();