diff options
author | Andrei Karas <akaras@inbox.ru> | 2014-04-03 12:25:27 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2014-04-03 12:25:27 +0300 |
commit | da87d0f7ce54182e3ea89a512ce3c24d5d06405e (patch) | |
tree | e08a5503e6ec45285b61ad06b56309184507087a /src/gui | |
parent | 3279c64d25124dbe759f7ba4fffe5554c3dd9e1e (diff) | |
download | mv-da87d0f7ce54182e3ea89a512ce3c24d5d06405e.tar.gz mv-da87d0f7ce54182e3ea89a512ce3c24d5d06405e.tar.bz2 mv-da87d0f7ce54182e3ea89a512ce3c24d5d06405e.tar.xz mv-da87d0f7ce54182e3ea89a512ce3c24d5d06405e.zip |
Remove duplicate code from minimap.
Diffstat (limited to 'src/gui')
-rw-r--r-- | src/gui/windows/minimap.cpp | 17 | ||||
-rw-r--r-- | src/gui/windows/minimap.h | 2 |
2 files changed, 8 insertions, 11 deletions
diff --git a/src/gui/windows/minimap.cpp b/src/gui/windows/minimap.cpp index 9be6d7304..70a2361cd 100644 --- a/src/gui/windows/minimap.cpp +++ b/src/gui/windows/minimap.cpp @@ -89,7 +89,12 @@ Minimap::~Minimap() config.setValue(getWindowName() + "Show", mShow); config.removeListeners(this); CHECKLISTENERS + deleteMapImage(); + delete2(mTextPopup); +} +void Minimap::deleteMapImage() +{ if (mMapImage) { if (mCustomMapImage) @@ -98,7 +103,6 @@ Minimap::~Minimap() mMapImage->decRef(); mMapImage = nullptr; } - delete2(mTextPopup); } void Minimap::setMap(const Map *const map) @@ -115,16 +119,7 @@ void Minimap::setMap(const Map *const map) } setCaption(caption); - - // Adapt the image - if (mMapImage) - { - if (mCustomMapImage) - delete mMapImage; - else - mMapImage->decRef(); - mMapImage = nullptr; - } + deleteMapImage(); if (map) { diff --git a/src/gui/windows/minimap.h b/src/gui/windows/minimap.h index e8fff5144..18a8a0a8b 100644 --- a/src/gui/windows/minimap.h +++ b/src/gui/windows/minimap.h @@ -73,6 +73,8 @@ class Minimap final : public Window, public ConfigListener void optionChanged(const std::string &name); private: + void deleteMapImage(); + float mWidthProportion; float mHeightProportion; Image *mMapImage; |