[mutter/wip/carlosg/input-thread: 104/159] backends: Remove x/y arguments from MetaCursorTracker::cursor-moved




commit ce47e60f2cc5e2b52237db33c458c6384a7311c5
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri Jul 10 00:40:55 2020 +0200

    backends: Remove x/y arguments from MetaCursorTracker::cursor-moved
    
    Make this signal a hint, the actual coordinates should be queried to the
    cursor tracker, the device, the seat... There's enough options.

 src/backends/meta-cursor-tracker.c                 | 13 +++----------
 src/backends/meta-screen-cast-area-stream-src.c    |  2 --
 src/backends/meta-screen-cast-monitor-stream-src.c |  2 --
 src/backends/meta-screen-cast-window-stream-src.c  |  2 --
 src/meson.build                                    |  8 --------
 src/meta-marshal.list                              |  1 -
 6 files changed, 3 insertions(+), 25 deletions(-)
---
diff --git a/src/backends/meta-cursor-tracker.c b/src/backends/meta-cursor-tracker.c
index c4f387f6ed..85deb798ba 100644
--- a/src/backends/meta-cursor-tracker.c
+++ b/src/backends/meta-cursor-tracker.c
@@ -37,7 +37,6 @@
 #include "cogl/cogl.h"
 #include "core/display-private.h"
 #include "clutter/clutter.h"
-#include "meta-marshal.h"
 #include "meta/main.h"
 #include "meta/util.h"
 
@@ -301,14 +300,8 @@ meta_cursor_tracker_class_init (MetaCursorTrackerClass *klass)
                                         G_TYPE_FROM_CLASS (klass),
                                         G_SIGNAL_RUN_LAST,
                                         0,
-                                        NULL, NULL,
-                                        meta_marshal_VOID__FLOAT_FLOAT,
-                                        G_TYPE_NONE, 2,
-                                        G_TYPE_FLOAT,
-                                        G_TYPE_FLOAT);
-  g_signal_set_va_marshaller (signals[CURSOR_MOVED],
-                              G_TYPE_FROM_CLASS (klass),
-                              meta_marshal_VOID__FLOAT_FLOATv);
+                                        NULL, NULL, NULL,
+                                        G_TYPE_NONE, 0);
 
   signals[VISIBILITY_CHANGED] = g_signal_new ("visibility-changed",
                                               G_TYPE_FROM_CLASS (klass),
@@ -459,7 +452,7 @@ meta_cursor_tracker_update_position (MetaCursorTracker *tracker,
   meta_cursor_renderer_set_position (cursor_renderer, new_x, new_y);
 
   if (position_changed)
-    g_signal_emit (tracker, signals[CURSOR_MOVED], 0, new_x, new_y);
+    g_signal_emit (tracker, signals[CURSOR_MOVED], 0);
 }
 
 void
diff --git a/src/backends/meta-screen-cast-area-stream-src.c b/src/backends/meta-screen-cast-area-stream-src.c
index c5521422c6..270bfa2a90 100644
--- a/src/backends/meta-screen-cast-area-stream-src.c
+++ b/src/backends/meta-screen-cast-area-stream-src.c
@@ -184,8 +184,6 @@ sync_cursor_state (MetaScreenCastAreaStreamSrc *area_src)
 
 static void
 cursor_moved (MetaCursorTracker           *cursor_tracker,
-              float                        x,
-              float                        y,
               MetaScreenCastAreaStreamSrc *area_src)
 {
   sync_cursor_state (area_src);
diff --git a/src/backends/meta-screen-cast-monitor-stream-src.c 
b/src/backends/meta-screen-cast-monitor-stream-src.c
index cb6e81b288..6c70bbc41b 100644
--- a/src/backends/meta-screen-cast-monitor-stream-src.c
+++ b/src/backends/meta-screen-cast-monitor-stream-src.c
@@ -221,8 +221,6 @@ sync_cursor_state (MetaScreenCastMonitorStreamSrc *monitor_src)
 
 static void
 cursor_moved (MetaCursorTracker              *cursor_tracker,
-              float                           x,
-              float                           y,
               MetaScreenCastMonitorStreamSrc *monitor_src)
 {
   sync_cursor_state (monitor_src);
diff --git a/src/backends/meta-screen-cast-window-stream-src.c 
b/src/backends/meta-screen-cast-window-stream-src.c
index d41b6c41c9..5f0f736afc 100644
--- a/src/backends/meta-screen-cast-window-stream-src.c
+++ b/src/backends/meta-screen-cast-window-stream-src.c
@@ -394,8 +394,6 @@ sync_cursor_state (MetaScreenCastWindowStreamSrc *window_src)
 
 static void
 cursor_moved (MetaCursorTracker             *cursor_tracker,
-              float                          x,
-              float                          y,
               MetaScreenCastWindowStreamSrc *window_src)
 {
   sync_cursor_state (window_src);
diff --git a/src/meson.build b/src/meson.build
index d81279a392..03faffdb7c 100644
--- a/src/meson.build
+++ b/src/meson.build
@@ -751,14 +751,6 @@ dbus_idle_monitor_built_sources = gnome.gdbus_codegen('meta-dbus-idle-monitor',
   )
 mutter_built_sources += dbus_idle_monitor_built_sources
 
-mutter_marshal = gnome.genmarshal('meta-marshal',
-    sources: ['meta-marshal.list'],
-    prefix: 'meta_marshal',
-    internal: true,
-    valist_marshallers: true,
-  )
-mutter_built_sources += mutter_marshal
-
 if have_profiler
   mutter_sources += [
     'backends/meta-profiler.c',


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]