Merge lp:~gero-bare/midori/midori-remove-dead-code into lp:midori

Proposed by Gero.Bare
Status: Merged
Approved by: Cris Dywan
Approved revision: 6826
Merged at revision: 6828
Proposed branch: lp:~gero-bare/midori/midori-remove-dead-code
Merge into: lp:midori
Diff against target: 55 lines (+1/-21)
2 files modified
midori/midori-browser.c (+1/-3)
midori/midori-preferences.c (+0/-18)
To merge this branch: bzr merge lp:~gero-bare/midori/midori-remove-dead-code
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Review via email: mp+240778@code.launchpad.net

Commit message

Remove dead code from browser and preferences

Description of the change

In order to compile Midori using clang first it's needed to fix some warnings.

This is dead code that prevents to clang to build properly Midori.

Of course those warnings can be disabled, but that does not address the issue of the dead code.

To post a comment you must log in.
Revision history for this message
Cris Dywan (kalikiana) wrote :

Makes sense.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-browser.c'
2--- midori/midori-browser.c 2014-09-06 09:02:47 +0000
3+++ midori/midori-browser.c 2014-11-05 21:15:41 +0000
4@@ -5448,7 +5448,6 @@
5 NULL, G_CALLBACK (_action_statusbar_activate),
6 FALSE },
7 };
8-static const guint toggle_entries_n = G_N_ELEMENTS (toggle_entries);
9
10 static const GtkRadioActionEntry encoding_entries[] =
11 {
12@@ -5481,7 +5480,6 @@
13 N_("Custom…"), "",
14 NULL, 1 },
15 };
16-static const guint encoding_entries_n = G_N_ELEMENTS (encoding_entries);
17
18 typedef struct {
19 MidoriBrowser* browser;
20@@ -7679,4 +7677,4 @@
21 g_return_if_fail (MIDORI_IS_BROWSER (browser));
22
23 g_signal_emit (browser, signals[QUIT], 0);
24-}
25\ No newline at end of file
26+}
27
28=== modified file 'midori/midori-preferences.c'
29--- midori/midori-preferences.c 2014-02-22 14:06:19 +0000
30+++ midori/midori-preferences.c 2014-11-05 21:15:41 +0000
31@@ -198,24 +198,6 @@
32 }
33 #endif
34
35-static inline void
36-midori_preferences_add_toolbutton (GtkWidget* toolbar,
37- GtkWidget** toolbutton,
38- const gchar* icon,
39- const gchar* label,
40- GtkWidget* page)
41-{
42-#if HAVE_OSX
43- *toolbutton = GTK_WIDGET (*toolbutton ? gtk_radio_tool_button_new_from_widget (
44- GTK_RADIO_TOOL_BUTTON (*toolbutton)) : gtk_radio_tool_button_new (NULL));
45- gtk_tool_button_set_label (GTK_TOOL_BUTTON (*toolbutton), label);
46- gtk_tool_button_set_stock_id (GTK_TOOL_BUTTON (*toolbutton), icon);
47- gtk_toolbar_insert (GTK_TOOLBAR (toolbar), GTK_TOOL_ITEM (*toolbutton), -1);
48- g_signal_connect (*toolbutton, "clicked",
49- G_CALLBACK (midori_preferences_toolbutton_clicked_cb), page);
50-#endif
51-}
52-
53 #if 0
54 static void
55 midori_preferences_list_dicts_cb (const gchar* lang_tag,

Subscribers

People subscribed via source and target branches

to all changes: