From 2991928437da9cf74c574949c31bd8ace2fecdde Mon Sep 17 00:00:00 2001 From: Paul Licameli Date: Tue, 12 Mar 2019 19:01:46 -0400 Subject: [PATCH] Remove stripping of / and & and space from align command ids ... ... this has been unnecessary since commit 9780c14 --- src/commands/CommandManager.cpp | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/src/commands/CommandManager.cpp b/src/commands/CommandManager.cpp index 11665b476..d958e99a4 100644 --- a/src/commands/CommandManager.cpp +++ b/src/commands/CommandManager.cpp @@ -961,14 +961,8 @@ CommandListEntry *CommandManager::NewIdentifier(const wxString & nameIn, // For key bindings for commands with a list, such as align, // the name in prefs is the category name plus the effect name. // This feature is not used for built-in effects. - if (multi) { - // The name needs to be clean for use by automation. - wxString cleanedName = wxString::Format(wxT("%s_%s"), name, nameSuffix); - cleanedName.Replace( "/", "" ); - cleanedName.Replace( "&", "" ); - cleanedName.Replace( " ", "" ); - name = cleanedName; - } + if (multi) + name = wxString::Format(wxT("%s_%s"), name, nameSuffix); // wxMac 2.5 and higher will do special things with the // Preferences, Exit (Quit), and About menu items,