Apply review suggestions

Author: lenemter

lib/Gestures/Gesture.vala

@@ -41,21 +41,21 @@ namespace Gala {
         MULTITASKING_VIEW,
         ZOOM,
         CUSTOM,
-        N_ACTIONS
-    }
+        N_ACTIONS;
 
-    public ModalActions gesture_action_to_modal_action (GestureAction gesture_action) {
-        switch (gesture_action) {
-            case SWITCH_WORKSPACE:
-                return SWITCH_WORKSPACE;
-            case SWITCH_WINDOWS:
-                return SWITCH_WINDOWS;
-            case MULTITASKING_VIEW:
-                return MULTITASKING_VIEW;
-            case ZOOM:
-                return ZOOM;
-            default:
-                return NONE;
+        public ModalActions to_modal_action () {
+            switch (this) {
+                case SWITCH_WORKSPACE:
+                    return ModalActions.SWITCH_WORKSPACE;
+                case SWITCH_WINDOWS:
+                    return ModalActions.SWITCH_WINDOWS;
+                case MULTITASKING_VIEW:
+                    return ModalActions.MULTITASKING_VIEW;
+                case ZOOM:
+                    return ModalActions.ZOOM;
+                default:
+                    return ModalActions.NONE;
+            }
         }
     }
 

lib/Gestures/GestureController.vala

@@ -157,7 +157,7 @@ public class Gala.GestureController : Object {
 
         var recognized_action = GestureSettings.get_action (gesture, out _action_info);
         recognizing = (
-            recognized_action == action && !wm.filter_action (gesture_action_to_modal_action (recognized_action)) ||
+            recognized_action == action && !wm.filter_action (recognized_action.to_modal_action ()) ||
             backend == scroll_backend && recognized_action == NONE
         );
 

lib/WindowManager.vala

@@ -38,11 +38,6 @@ namespace Gala {
         MEDIA_KEYS
     }
 
-    /**
-     * Function that should return true if the given shortcut should be blocked.
-     */
-    public delegate bool KeybindingFilter (Meta.KeyBinding binding);
-
     /**
      * A minimal class mostly used to identify your call to {@link WindowManager.push_modal} and used
      * to end your modal mode again with {@link WindowManager.pop_modal}