all repos — fluxbox @ 0b6c860fa0067c73216209878ed1ae46f541f571

custom fork of the fluxbox windowmanager

delete config menu first at shutdown
Henrik Kinnunen fluxgen at fluxbox dot org
commit

0b6c860fa0067c73216209878ed1ae46f541f571

parent

97e08c3853a77571f4d51b481581877db2250555

1 files changed, 11 insertions(+), 3 deletions(-)

jump to
M src/Screen.ccsrc/Screen.cc

@@ -504,7 +504,9 @@ BScreen::~BScreen() {

if (! managed) return; - + + m_configmenu.reset(0); + m_toolbar.reset(0); FbTk::EventManager *evm = FbTk::EventManager::instance();

@@ -573,7 +575,12 @@ }

// slit must be destroyed before headAreas (Struts) m_slit.reset(0); + + delete m_rootmenu.release(); + delete m_workspacemenu.release(); + delete m_windowmenu.release(); + // TODO fluxgen: check if this is the right place for (size_t i = 0; i < m_head_areas.size(); i++) delete m_head_areas[i];

@@ -1469,7 +1476,8 @@ }

void BScreen::addConfigMenu(const FbTk::FbString &label, FbTk::Menu &menu) { m_configmenu_list.push_back(make_pair(label, &menu)); - setupConfigmenu(*m_configmenu.get()); + if (m_configmenu.get()) + setupConfigmenu(*m_configmenu.get()); } void BScreen::removeConfigMenu(FbTk::Menu &menu) {

@@ -1480,7 +1488,7 @@ FbTk::Select2nd<Configmenus::value_type>()));

if (erase_it != m_configmenu_list.end()) m_configmenu_list.erase(erase_it); - if (!isShuttingdown()) + if (!isShuttingdown() && m_configmenu.get()) setupConfigmenu(*m_configmenu.get()); }