Drop gdk_app_launch_context_set_screen
authorMatthias Clasen <mclasen@redhat.com>
Tue, 31 Oct 2017 03:01:34 +0000 (23:01 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Tue, 31 Oct 2017 03:01:34 +0000 (23:01 -0400)
This function is not necessary, since we already know the display.

gdk/gdkapplaunchcontext.c
gdk/gdkapplaunchcontextprivate.h
gdk/wayland/gdkapplaunchcontext-wayland.c
gdk/x11/gdkapplaunchcontext-x11.c

index ab19967c45f68f6b0478ded3874df2ff98e00096..4f693e0995aa4e13c9d5846ed34583a37a45b4dd 100644 (file)
@@ -42,7 +42,7 @@
  *
  * context = gdk_display_get_app_launch_context (display);
  *
- * gdk_app_launch_context_set_screen (screen);
+ * gdk_app_launch_context_set_display (display);
  * gdk_app_launch_context_set_timestamp (event->time);
  *
  * if (!g_app_info_launch_default_for_uri ("http://www.gtk.org", context, &error))
@@ -143,9 +143,6 @@ gdk_app_launch_context_finalize (GObject *object)
   if (context->display)
     g_object_unref (context->display);
 
-  if (context->screen)
-    g_object_unref (context->screen);
-
   if (context->icon)
     g_object_unref (context->icon);
 
@@ -170,39 +167,6 @@ gdk_app_launch_context_get_display (GAppLaunchContext *context,
   return g_strdup (gdk_display_get_name (display));
 }
 
-/**
- * gdk_app_launch_context_set_screen:
- * @context: a #GdkAppLaunchContext
- * @screen: a #GdkScreen
- *
- * Sets the screen on which applications will be launched when
- * using this context. See also gdk_app_launch_context_set_display().
- *
- * If both @screen and @display are set, the @screen takes priority.
- * If neither @screen or @display are set, the default screen and
- * display are used.
- *
- * Since: 2.14
- */
-void
-gdk_app_launch_context_set_screen (GdkAppLaunchContext *context,
-                                   GdkScreen           *screen)
-{
-  g_return_if_fail (GDK_IS_APP_LAUNCH_CONTEXT (context));
-  g_return_if_fail (screen == NULL || GDK_IS_SCREEN (screen));
-
-  g_return_if_fail (screen == NULL || gdk_screen_get_display (screen) == context->display);
-
-  if (context->screen)
-    {
-      g_object_unref (context->screen);
-      context->screen = NULL;
-    }
-
-  if (screen)
-    context->screen = g_object_ref (screen);
-}
-
 /**
  * gdk_app_launch_context_set_desktop:
  * @context: a #GdkAppLaunchContext
index 2e0e6e0ea06c3052f15369bbc891dd4f6f5b6afb..bfb392598aaf60d6eaa3600a17e23cebc546d4b5 100644 (file)
@@ -36,7 +36,6 @@ struct _GdkAppLaunchContext
   GAppLaunchContext parent_instance;
 
   GdkDisplay *display;
-  GdkScreen *screen;
   gint workspace;
   guint32 timestamp;
   GIcon *icon;
index b69cc31fce50625b2781a6ea78d94a53b8d6c024..943a1c918518e0561b81d25eb530c5319433b27c 100644 (file)
@@ -26,7 +26,6 @@
 #include "gdkwayland.h"
 #include "gdkprivate-wayland.h"
 #include "gdkapplaunchcontextprivate.h"
-#include "gdkscreen.h"
 #include "gdkinternals.h"
 #include "gdkintl.h"
 
index b7fa648e23120656d98b42f79816e7a6a8b5a51c..7abb0a1da026bd51cdae18cd6094ecca093ee957 100644 (file)
@@ -282,10 +282,7 @@ gdk_x11_app_launch_context_get_startup_notify_id (GAppLaunchContext *context,
   ctx = GDK_APP_LAUNCH_CONTEXT (context);
 
   display = ctx->display;
-  if (ctx->screen)
-    screen = ctx->screen;
-  else
-    screen = gdk_display_get_default_screen (ctx->display);
+  screen = gdk_display_get_default_screen (ctx->display);
 
   fileinfo = NULL;
 
@@ -402,10 +399,7 @@ gdk_x11_app_launch_context_launch_failed (GAppLaunchContext *context,
 
   ctx = GDK_APP_LAUNCH_CONTEXT (context);
 
-  if (ctx->screen)
-    screen = ctx->screen;
-  else
-    screen = gdk_display_get_default_screen (ctx->display);
+  screen = gdk_display_get_default_screen (ctx->display);
 
   data = g_object_get_data (G_OBJECT (screen), "appinfo-startup-data");