From 9c5799c842fc61a9632150cd05f9eb6cd2da78ea Mon Sep 17 00:00:00 2001 From: Maarten ter Huurne Date: Sat, 19 Jun 2010 04:51:53 +0200 Subject: [PATCH] Add explicit destructor to BrowseDialog. There is no point adding virtual destructors to the subclasses if the superclass has a non-virtual default destructor... --- src/browsedialog.cpp | 4 ++++ src/browsedialog.h | 1 + 2 files changed, 5 insertions(+) diff --git a/src/browsedialog.cpp b/src/browsedialog.cpp index 69c188e..e17f27a 100644 --- a/src/browsedialog.cpp +++ b/src/browsedialog.cpp @@ -30,6 +30,10 @@ BrowseDialog::BrowseDialog(GMenu2X *gmenu2x, const string &title, iconFile = gmenu2x->sc.skinRes("imgs/file.png"); } +BrowseDialog::~BrowseDialog() +{ +} + bool BrowseDialog::exec() { if (!fl) diff --git a/src/browsedialog.h b/src/browsedialog.h index b97d117..f7835a2 100644 --- a/src/browsedialog.h +++ b/src/browsedialog.h @@ -35,6 +35,7 @@ using std::vector; class BrowseDialog : protected Dialog { protected: BrowseDialog(GMenu2X *gmenu2x, const string &title, const string &subtitle); + virtual ~BrowseDialog(); void setPath(const string &path) { fl->setPath(path);