smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-07 15:25:01 +01:00
Code Issues Projects Releases Wiki Activity

Separate set_toggleaction_state () API

Browse source 
Prepare for a method consistent with access_action():
* separate group + action names
* no action string parsing overhead.
* no fatal, abort () call for invalid actions
This commit is contained in:
Robin Gareus 2017-08-09 13:33:22 +02:00
parent e1a29c4e46
commit bbc8c1354e
2 changed files with 26 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

29
libs/gtkmm2ext/actions.cc
View file

@ -220,19 +220,19 @@ ActionManager::set_sensitive (vector<RefPtr<Action> >& actions, bool state)
} }
void void
ActionManager::check_toggleaction (string n) ActionManager::check_toggleaction (const string& n)
{ {
set_toggleaction_state (n, true); set_toggleaction_state (n, true);
} }
void void
ActionManager::uncheck_toggleaction (string n) ActionManager::uncheck_toggleaction (const string& n)
{ {
set_toggleaction_state (n, false); set_toggleaction_state (n, false);
} }
void void
ActionManager::set_toggleaction_state (string n, bool s) ActionManager::set_toggleaction_state (const string& n, bool s)
{ {
char const * name = n.c_str (); char const * name = n.c_str ();
@ -252,18 +252,27 @@ ActionManager::set_toggleaction_state (string n, bool s)
group_name[len] = '\0'; group_name[len] = '\0';
const char* action_name = last_slash + 1; const char* action_name = last_slash + 1;
if (!set_toggleaction_state (group_name, action_name, s)) {
RefPtr<Action> act = get_action (group_name, action_name); error << string_compose (_("Unknown action name: %1/%2"), group_name, action_name) << endmsg;
if (act) {
RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
tact->set_active (s);
} else {
error << string_compose (_("Unknown action name: %1"), name) << endmsg;
} }
delete [] group_name; delete [] group_name;
} }
bool
ActionManager::set_toggleaction_state (const char* group_name, const char* action_name, bool s)
{
RefPtr<Action> act = get_action (group_name, action_name);
if (act) {
RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
if (tact) {
tact->set_active (s);
return true;
}
}
return false;
}
void void
ActionManager::do_action (const char* group, const char*action) ActionManager::do_action (const char* group, const char*action)
{ {

8
libs/gtkmm2ext/gtkmm2ext/actions.h
View file

@ -49,10 +49,10 @@ namespace ActionManager {
LIBGTKMM2EXT_API extern void do_action (const char* group, const char* name); LIBGTKMM2EXT_API extern void do_action (const char* group, const char* name);
LIBGTKMM2EXT_API extern void set_toggle_action (const char* group, const char* name, bool); LIBGTKMM2EXT_API extern void set_toggle_action (const char* group, const char* name, bool);
LIBGTKMM2EXT_API extern void check_toggleaction (std::string); LIBGTKMM2EXT_API extern void check_toggleaction (const std::string&);
LIBGTKMM2EXT_API extern void uncheck_toggleaction (std::string); LIBGTKMM2EXT_API extern void uncheck_toggleaction (const std::string&);
LIBGTKMM2EXT_API extern void set_toggleaction_state (std::string, bool); LIBGTKMM2EXT_API extern void set_toggleaction_state (const std::string&, bool);
LIBGTKMM2EXT_API extern bool set_toggleaction_state (const char*, const char*, bool);
LIBGTKMM2EXT_API extern void save_action_states (); LIBGTKMM2EXT_API extern void save_action_states ();
LIBGTKMM2EXT_API extern void enable_active_actions (); LIBGTKMM2EXT_API extern void enable_active_actions ();