Merge lp:~charlesk/indicator-appmenu/nogtk2 into lp:indicator-appmenu/12.10

Proposed by Charles Kerr
Status: Merged
Approved by: Lars Karlitski
Approved revision: 205
Merged at revision: 205
Proposed branch: lp:~charlesk/indicator-appmenu/nogtk2
Merge into: lp:indicator-appmenu/12.10
Diff against target: 125 lines (+8/-52)
4 files modified
configure.ac (+7/-28)
src/gtkmodelmenuitem.c (+0/-13)
src/indicator-appmenu.c (+1/-5)
src/window-menu-model.c (+0/-6)
To merge this branch: bzr merge lp:~charlesk/indicator-appmenu/nogtk2
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
Review via email: mp+105013@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Lars Karlitski (larsu) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'configure.ac'
2--- configure.ac 2012-04-05 16:04:11 +0000
3+++ configure.ac 2012-05-08 00:40:26 +0000
4@@ -57,39 +57,18 @@
5 # Dependencies
6 ###########################
7
8-GTK_REQUIRED_VERSION=2.12
9-GTK3_REQUIRED_VERSION=3.0
10+GTK_REQUIRED_VERSION=3.0
11 GIO_REQUIRED_VERSION=2.26
12 INDICATOR_REQUIRED_VERSION=0.3.90
13 DBUSMENUGTK_REQUIRED_VERSION=0.5.90
14 BAMF_REQUIRED_VERSION=0.2.53
15
16-AC_ARG_WITH([gtk],
17- [AS_HELP_STRING([--with-gtk],
18- [Which version of gtk to use @<:@default=3@:>@])],
19- [],
20- [with_gtk=3])
21-
22-AS_IF([test "x$with_gtk" = x3],
23- [PKG_CHECK_MODULES(INDICATOR, gtk+-3.0 >= $GTK3_REQUIRED_VERSION
24- gio-2.0 >= $GIO_REQUIRED_VERSION
25- indicator3-0.4 >= $INDICATOR_REQUIRED_VERSION
26- dbusmenu-gtk3-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION
27- libbamf3 >= $BAMF_REQUIRED_VERSION)
28- AC_DEFINE(HAVE_GTK3, 1, [whether gtk3 is available])
29- ],
30- [test "x$with_gtk" = x2],
31- [PKG_CHECK_MODULES(INDICATOR, gtk+-2.0 >= $GTK_REQUIRED_VERSION
32- gio-2.0 >= $GIO_REQUIRED_VERSION
33- indicator-0.4 >= $INDICATOR_REQUIRED_VERSION
34- dbusmenu-gtk-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION
35- libbamf >= $BAMF_REQUIRED_VERSION)
36- ],
37- [AC_MSG_FAILURE([Value for --with-gtk was neither 2 nor 3])]
38-)
39-
40-PKG_CHECK_MODULES(gtk, gtk+-3.0 >= $GTK3_REQUIRED_VERSION)
41-
42+PKG_CHECK_MODULES(gtk, gtk+-3.0 >= $GTK_REQUIRED_VERSION)
43+PKG_CHECK_MODULES(INDICATOR, gtk+-3.0 >= $GTK_REQUIRED_VERSION
44+ gio-2.0 >= $GIO_REQUIRED_VERSION
45+ indicator3-0.4 >= $INDICATOR_REQUIRED_VERSION
46+ dbusmenu-gtk3-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION
47+ libbamf3 >= $BAMF_REQUIRED_VERSION)
48 AC_SUBST(INDICATOR_CFLAGS)
49 AC_SUBST(INDICATOR_LIBS)
50
51
52=== modified file 'src/gtkmodelmenuitem.c'
53--- src/gtkmodelmenuitem.c 2012-03-21 15:42:18 +0000
54+++ src/gtkmodelmenuitem.c 2012-05-08 00:40:26 +0000
55@@ -63,7 +63,6 @@
56 *requisition = 0;
57 }
58
59-#if HAVE_GTK3
60 static void
61 gtk_model_menu_item_draw_indicator (GtkCheckMenuItem *check_item,
62 cairo_t *cr)
63@@ -74,18 +73,6 @@
64 GTK_CHECK_MENU_ITEM_CLASS (gtk_model_menu_item_parent_class)
65 ->draw_indicator (check_item, cr);
66 }
67-#else
68-static void
69-gtk_model_menu_item_draw_indicator (GtkCheckMenuItem *check_item,
70- GdkRectangle *area)
71-{
72- GtkModelMenuItem *item = GTK_MODEL_MENU_ITEM (check_item);
73-
74- if (item->has_indicator)
75- GTK_CHECK_MENU_ITEM_CLASS (gtk_model_menu_item_parent_class)
76- ->draw_indicator (check_item, area);
77-}
78-#endif
79
80 static void
81 gtk_model_menu_item_set_active (GtkModelMenuItem *item,
82
83=== modified file 'src/indicator-appmenu.c'
84--- src/indicator-appmenu.c 2012-04-04 20:23:20 +0000
85+++ src/indicator-appmenu.c 2012-05-08 00:40:26 +0000
86@@ -532,12 +532,8 @@
87 False,
88 SubstructureRedirectMask | SubstructureNotifyMask,
89 &xev);
90- gdk_flush ();
91-#if GTK_CHECK_VERSION(3, 0, 0)
92+ gdk_flush ();
93 gdk_error_trap_pop_ignored ();
94-#else
95- gdk_error_trap_pop ();
96-#endif
97
98 return;
99 }
100
101=== modified file 'src/window-menu-model.c'
102--- src/window-menu-model.c 2012-03-21 14:05:28 +0000
103+++ src/window-menu-model.c 2012-05-08 00:40:26 +0000
104@@ -386,9 +386,7 @@
105 static void
106 item_inserted_cb (GtkContainer *menu,
107 GtkWidget *widget,
108-#ifdef HAVE_GTK3
109 gint position,
110-#endif
111 gpointer data)
112 {
113 if (g_object_get_data(G_OBJECT(widget), ENTRY_DATA) == NULL) {
114@@ -422,11 +420,7 @@
115 g_object_ref_sink(menu->priv->win_menu);
116
117 menu->priv->win_menu_insert = g_signal_connect(G_OBJECT (menu->priv->win_menu),
118-#ifdef HAVE_GTK3
119 "insert",
120-#else
121- "child-added",
122-#endif
123 G_CALLBACK (item_inserted_cb),
124 menu);
125 menu->priv->win_menu_remove = g_signal_connect (G_OBJECT (menu->priv->win_menu),

Subscribers

People subscribed via source and target branches