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
=== modified file 'midori/midori-browser.c'
--- midori/midori-browser.c 2014-09-06 09:02:47 +0000
+++ midori/midori-browser.c 2014-11-05 21:15:41 +0000
@@ -5448,7 +5448,6 @@
5448 NULL, G_CALLBACK (_action_statusbar_activate),5448 NULL, G_CALLBACK (_action_statusbar_activate),
5449 FALSE },5449 FALSE },
5450};5450};
5451static const guint toggle_entries_n = G_N_ELEMENTS (toggle_entries);
54525451
5453static const GtkRadioActionEntry encoding_entries[] =5452static const GtkRadioActionEntry encoding_entries[] =
5454{5453{
@@ -5481,7 +5480,6 @@
5481 N_("Custom…"), "",5480 N_("Custom…"), "",
5482 NULL, 1 },5481 NULL, 1 },
5483};5482};
5484static const guint encoding_entries_n = G_N_ELEMENTS (encoding_entries);
54855483
5486typedef struct {5484typedef struct {
5487 MidoriBrowser* browser;5485 MidoriBrowser* browser;
@@ -7679,4 +7677,4 @@
7679 g_return_if_fail (MIDORI_IS_BROWSER (browser));7677 g_return_if_fail (MIDORI_IS_BROWSER (browser));
76807678
7681 g_signal_emit (browser, signals[QUIT], 0);7679 g_signal_emit (browser, signals[QUIT], 0);
7682}
7683\ No newline at end of file7680\ No newline at end of file
7681}
76847682
=== modified file 'midori/midori-preferences.c'
--- midori/midori-preferences.c 2014-02-22 14:06:19 +0000
+++ midori/midori-preferences.c 2014-11-05 21:15:41 +0000
@@ -198,24 +198,6 @@
198}198}
199#endif199#endif
200200
201static inline void
202midori_preferences_add_toolbutton (GtkWidget* toolbar,
203 GtkWidget** toolbutton,
204 const gchar* icon,
205 const gchar* label,
206 GtkWidget* page)
207{
208#if HAVE_OSX
209 *toolbutton = GTK_WIDGET (*toolbutton ? gtk_radio_tool_button_new_from_widget (
210 GTK_RADIO_TOOL_BUTTON (*toolbutton)) : gtk_radio_tool_button_new (NULL));
211 gtk_tool_button_set_label (GTK_TOOL_BUTTON (*toolbutton), label);
212 gtk_tool_button_set_stock_id (GTK_TOOL_BUTTON (*toolbutton), icon);
213 gtk_toolbar_insert (GTK_TOOLBAR (toolbar), GTK_TOOL_ITEM (*toolbutton), -1);
214 g_signal_connect (*toolbutton, "clicked",
215 G_CALLBACK (midori_preferences_toolbutton_clicked_cb), page);
216#endif
217}
218
219#if 0201#if 0
220static void202static void
221midori_preferences_list_dicts_cb (const gchar* lang_tag,203midori_preferences_list_dicts_cb (const gchar* lang_tag,

Subscribers

People subscribed via source and target branches

to all changes: