ardour/ardour
ardour/ardour
13
1
Fork 0
Code Pull Requests Releases Activity

Fixes minor memory leak found in color_theme_manager.cpp:326

Browse Source
This commit is contained in:
Nathan Stewart 2016-10-11 11:07:17 -04:00 committed by Paul Davis
parent ca98c7a051
commit 114bc97f94
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
gtk2_ardour/color_theme_manager.cc
View File

@ -165,6 +165,16 @@ ColorThemeManager::ColorThemeManager ()
show_all ();
}
ColorThemeManager::~ColorThemeManager ()
{
if (palette_group) {
palette_group->clear (true);
delete palette_group;
}
}
void
ColorThemeManager::setup_modifiers ()
{
@ -311,7 +321,6 @@ ColorThemeManager::build_palette_canvas (ArdourCanvas::Canvas& canvas, ArdourCan
for (uint32_t y = 0; y < height - box_size && color_num < color_limit; y += box_size) {
for (uint32_t x = 0; x < width - box_size && color_num < color_limit; x += box_size) {
ArdourCanvas::Rectangle* r = new ArdourCanvas::Rectangle (&group, ArdourCanvas::Rect (x, y, x + box_size, y + box_size));
string name = nc[color_num++].name;
UIConfiguration::Colors::iterator c = colors.find (name);

1
gtk2_ardour/color_theme_manager.h
View File

@ -43,6 +43,7 @@ class ColorThemeManager : public Gtk::VBox, public OptionEditorComponent
{
public:
ColorThemeManager();
~ColorThemeManager();
void reset_canvas_colors();
void on_color_theme_changed ();