[gtk+] Drop the GDK_PRIVATE_CALL mechanism



commit 3414f10badc821b7f727e3aad21735d2c4a7596c
Author: Matthias Clasen <mclasen redhat com>
Date:   Wed Nov 23 19:31:16 2016 -0500

    Drop the GDK_PRIVATE_CALL mechanism
    
    We don't need to do this extra complication anymore, since
    we're now living inside a single shared library.

 gdk/Makefile.am   |    1 -
 gdk/gdk-private.c |   22 ----------------------
 gdk/gdk-private.h |   36 ------------------------------------
 gtk/gtkmain.c     |   12 ++++++------
 gtk/gtkmenu.c     |   14 +++++++-------
 gtk/gtkwindow.c   |    6 +++---
 6 files changed, 16 insertions(+), 75 deletions(-)
---
diff --git a/gdk/Makefile.am b/gdk/Makefile.am
index 9549144..f4522c0 100644
--- a/gdk/Makefile.am
+++ b/gdk/Makefile.am
@@ -121,7 +121,6 @@ gdk_private_headers =                               \
        gdkx.h
 
 gdk_c_sources =                                \
-       gdk-private.c                           \
        gdk.c                                   \
        gdkapplaunchcontext.c                   \
        gdkcairo.c                              \
diff --git a/gdk/gdk-private.h b/gdk/gdk-private.h
index ba16b92..a0d102f 100644
--- a/gdk/gdk-private.h
+++ b/gdk/gdk-private.h
@@ -4,8 +4,6 @@
 #include <gdk/gdk.h>
 #include "gdk/gdkinternals.h"
 
-#define GDK_PRIVATE_CALL(symbol)        (gdk__private__ ()->symbol)
-
 GdkDisplay *    gdk_display_open_default        (void);
 
 gboolean        gdk_device_grab_info            (GdkDisplay  *display,
@@ -35,38 +33,4 @@ void            gdk_window_move_to_rect         (GdkWindow          *window,
                                                  gint                rect_anchor_dx,
                                                  gint                rect_anchor_dy);
 
-typedef struct {
-  /* add all private functions here, initialize them in gdk-private.c */
-  gboolean (* gdk_device_grab_info) (GdkDisplay  *display,
-                                     GdkDevice   *device,
-                                     GdkWindow  **grab_window,
-                                     gboolean    *owner_events);
-
-  GdkDisplay *(* gdk_display_open_default) (void);
-
-  void (* gdk_add_option_entries) (GOptionGroup *group);
-  void (* gdk_pre_parse) (void);
-
-  GdkGLFlags (* gdk_gl_get_flags) (void);
-  void       (* gdk_gl_set_flags) (GdkGLFlags flags);
-
-  void (* gdk_window_freeze_toplevel_updates) (GdkWindow *window);
-  void (* gdk_window_thaw_toplevel_updates) (GdkWindow *window);
-
-  GdkRenderingMode (* gdk_display_get_rendering_mode) (GdkDisplay       *display);
-  void             (* gdk_display_set_rendering_mode) (GdkDisplay       *display,
-                                                       GdkRenderingMode  mode);
-
-  void (* gdk_window_move_to_rect) (GdkWindow          *window,
-                                    const GdkRectangle *rect,
-                                    GdkGravity          rect_anchor,
-                                    GdkGravity          window_anchor,
-                                    GdkAnchorHints      anchor_hints,
-                                    gint                rect_anchor_dx,
-                                    gint                rect_anchor_dy);
-} GdkPrivateVTable;
-
-GDK_AVAILABLE_IN_ALL
-GdkPrivateVTable *      gdk__private__  (void);
-
 #endif /* __GDK__PRIVATE_H__ */
diff --git a/gtk/gtkmain.c b/gtk/gtkmain.c
index e7dca63..25e2047 100644
--- a/gtk/gtkmain.c
+++ b/gtk/gtkmain.c
@@ -632,7 +632,7 @@ do_pre_parse_initialization (int    *argc,
   if (_gtk_module_has_mixed_deps (NULL))
     g_error ("GTK+ 2.x symbols detected. Using GTK+ 2.x and GTK+ 3 in the same process is not supported");
 
-  GDK_PRIVATE_CALL (gdk_pre_parse) ();
+  gdk_pre_parse ();
   gdk_event_handler_set ((GdkEventFunc)gtk_main_do_event, NULL, NULL);
 
 #ifdef G_ENABLE_DEBUG
@@ -775,7 +775,7 @@ post_parse_hook (GOptionContext *context,
   
   if (info->open_default_display)
     {
-      if (GDK_PRIVATE_CALL (gdk_display_open_default) () == NULL)
+      if (gdk_display_open_default () == NULL)
         {
           const char *display_name = gdk_get_display_arg_name ();
           g_set_error (error,
@@ -893,7 +893,7 @@ gtk_get_option_group (gboolean open_default_display)
   group = g_option_group_new ("gtk", _("GTK+ Options"), _("Show GTK+ Options"), info, g_free);
   g_option_group_set_parse_hooks (group, pre_parse_hook, post_parse_hook);
 
-  GDK_PRIVATE_CALL (gdk_add_option_entries) (group);
+  gdk_add_option_entries (group);
   g_option_group_add_entries (group, gtk_args);
   g_option_group_set_translation_domain (group, GETTEXT_PACKAGE);
   
@@ -965,7 +965,7 @@ gtk_init_with_args (gint                 *argc,
     return FALSE;
 
 done:
-  return GDK_PRIVATE_CALL (gdk_display_open_default) () != NULL;
+  return gdk_display_open_default () != NULL;
 }
 
 
@@ -1056,7 +1056,7 @@ gtk_init_check (int    *argc,
   if (!gtk_parse_args (argc, argv))
     return FALSE;
 
-  ret = GDK_PRIVATE_CALL (gdk_display_open_default) () != NULL;
+  ret = gdk_display_open_default () != NULL;
 
   if (gtk_get_debug_flags () & GTK_DEBUG_INTERACTIVE)
     gtk_window_set_interactive_debugging (TRUE);
@@ -1525,7 +1525,7 @@ rewrite_event_for_grabs (GdkEvent *event)
       display = gdk_window_get_display (event->any.window);
       device = gdk_event_get_device (event);
 
-      if (!GDK_PRIVATE_CALL (gdk_device_grab_info) (display, device, &grab_window, &owner_events) ||
+      if (!gdk_device_grab_info (display, device, &grab_window, &owner_events) ||
           !owner_events)
         return NULL;
       break;
diff --git a/gtk/gtkmenu.c b/gtk/gtkmenu.c
index 514dd67..7d2c3c9 100644
--- a/gtk/gtkmenu.c
+++ b/gtk/gtkmenu.c
@@ -4541,13 +4541,13 @@ gtk_menu_position (GtkMenu  *menu,
     g_signal_connect (toplevel, "moved-to-rect", G_CALLBACK (moved_to_rect_cb),
                       menu);
 
-  GDK_PRIVATE_CALL (gdk_window_move_to_rect) (toplevel,
-                                              &rect,
-                                              rect_anchor,
-                                              menu_anchor,
-                                              anchor_hints,
-                                              rect_anchor_dx,
-                                              rect_anchor_dy);
+  gdk_window_move_to_rect (toplevel,
+                           &rect,
+                           rect_anchor,
+                           menu_anchor,
+                           anchor_hints,
+                           rect_anchor_dx,
+                           rect_anchor_dy);
 }
 
 static void
diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
index 8921369..2715d0c 100644
--- a/gtk/gtkwindow.c
+++ b/gtk/gtkwindow.c
@@ -6130,7 +6130,7 @@ gtk_window_unmap (GtkWidget *widget)
   while (priv->configure_request_count > 0)
     {
       priv->configure_request_count--;
-      GDK_PRIVATE_CALL (gdk_window_thaw_toplevel_updates) (_gtk_widget_get_window (widget));
+      gdk_window_thaw_toplevel_updates (_gtk_widget_get_window (widget));
     }
   priv->configure_notify_received = FALSE;
 
@@ -7444,7 +7444,7 @@ gtk_window_configure_event (GtkWidget         *widget,
     {
       priv->configure_request_count -= 1;
 
-      GDK_PRIVATE_CALL (gdk_window_thaw_toplevel_updates) (_gtk_widget_get_window (widget));
+      gdk_window_thaw_toplevel_updates (_gtk_widget_get_window (widget));
     }
 
   /*
@@ -9160,7 +9160,7 @@ gtk_window_move_resize (GtkWindow *window)
          /* Increment the number of have-not-yet-received-notify requests */
          priv->configure_request_count += 1;
 
-          GDK_PRIVATE_CALL (gdk_window_freeze_toplevel_updates) (gdk_window);
+          gdk_window_freeze_toplevel_updates (gdk_window);
 
          /* for GTK_RESIZE_QUEUE toplevels, we are now awaiting a new
           * configure event in response to our resizing request.


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