From 91f651c6f6f0ee7fdff1d49e36161d4e76e16e02 Mon Sep 17 00:00:00 2001 From: Steve Zesch Date: Sat, 4 Aug 2012 19:01:34 -0400 Subject: Micro-optimization. do_multimedia_player_action did nothing more than call msd_media_player_key_pressed, which was pointless overhead. --- plugins/media-keys/msd-media-keys-manager.c | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) (limited to 'plugins/media-keys') diff --git a/plugins/media-keys/msd-media-keys-manager.c b/plugins/media-keys/msd-media-keys-manager.c index d75c031..f65596d 100644 --- a/plugins/media-keys/msd-media-keys-manager.c +++ b/plugins/media-keys/msd-media-keys-manager.c @@ -979,13 +979,6 @@ msd_media_player_key_pressed (MsdMediaKeysManager *manager, return !have_listeners; } -static gboolean -do_multimedia_player_action (MsdMediaKeysManager *manager, - const char *key) -{ - return msd_media_player_key_pressed (manager, key); -} - static gboolean do_action (MsdMediaKeysManager *manager, int type) @@ -1059,19 +1052,19 @@ do_action (MsdMediaKeysManager *manager, g_free (cmd); break; case PLAY_KEY: - return do_multimedia_player_action (manager, "Play"); + return msd_media_player_key_pressed (manager, "Play"); break; case PAUSE_KEY: - return do_multimedia_player_action (manager, "Pause"); + return msd_media_player_key_pressed (manager, "Pause"); break; case STOP_KEY: - return do_multimedia_player_action (manager, "Stop"); + return msd_media_player_key_pressed (manager, "Stop"); break; case PREVIOUS_KEY: - return do_multimedia_player_action (manager, "Previous"); + return msd_media_player_key_pressed (manager, "Previous"); break; case NEXT_KEY: - return do_multimedia_player_action (manager, "Next"); + return msd_media_player_key_pressed (manager, "Next"); break; default: g_assert_not_reached (); -- cgit v1.2.1