From 76d79985cd272ef8501b69f061574d85b45df659 Mon Sep 17 00:00:00 2001 From: Rob McMullen Date: Sun, 22 Nov 2015 23:18:01 -0800 Subject: [PATCH] Fix for OS X crash on exit: moved FontMapping* action to separate menu Still don't know why, but including those actions in the same menu as the ANTIC font items crashed. I thought it was because I was using the same SMenu id which resulted in getting the actions added into the Antic Mode menu, but putting the Groups into the same SMenu still crashed. I thought it might be something to do with the Group ids being reused, so I changed the "a1", "a2", etc to something else, but that didn't work either. Ultimately, moving the FontMapping* stuff into its own SMenu with a different id worked. Maybe because there were multiple generic Actions in different groups that got added into the same menu? Or because they were both radio buttons? --- omnimon/_omnimon_version.py | 2 +- omnimon/tasks/hex_edit/task.py | 4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/omnimon/_omnimon_version.py b/omnimon/_omnimon_version.py index 1663d823..181e9f09 100644 --- a/omnimon/_omnimon_version.py +++ b/omnimon/_omnimon_version.py @@ -1 +1 @@ -version = "0.1.1" +version = "0.2.0" diff --git a/omnimon/tasks/hex_edit/task.py b/omnimon/tasks/hex_edit/task.py index 26b1b1eb..c2a7e768 100644 --- a/omnimon/tasks/hex_edit/task.py +++ b/omnimon/tasks/hex_edit/task.py @@ -132,12 +132,10 @@ def get_actions(self, location, menu_name, group_name): Group( *font_mapping_actions, id="a2", separator=True), - id='FontChoiceSubmenu2', separator=True, name="Antic Font Mapping"), - SMenu( Group( FontMappingWidthAction(), id="a3", separator=True), - id='FontChoiceSubmenu2', separator=True, name="Antic Font Width"), + id='FontChoiceSubmenu2a1', separator=True, name="Char Map"), SMenu( Group( UseColorsAction(name="Powerup Colors", colors=colors.powerup_colors()),