commit 5e431cf1de79a02fe3f3c2fffdf0423b5d66fea1
author Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com> Thu Dec 24 14:09:01 2015 -0500
committer Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com> Mon Jan 04 14:01:29 2016 -0400
tree 4c722852bd3e844464c617ec17226310de75e9f5
parent e97b9404c5b1b1c8fbc60e7ac95e71b664f5f1fe

main window: prevent video-double-clicked warnings

Change-Id: I2faa43e7291fbb686d17a0ca357480d2e2754798
Tuleap: #211

src/ringmainwindow.cpp

diff --git a/src/ringmainwindow.cpp b/src/ringmainwindow.cpp
index 5045c93..a60ee99 100644
--- a/src/ringmainwindow.cpp
+++ b/src/ringmainwindow.cpp
@@ -172,7 +172,7 @@
 }
 
 static void
-video_double_clicked(CurrentCallView *view, RingMainWindow *self)
+video_double_clicked(G_GNUC_UNUSED CurrentCallView *view, RingMainWindow *self)
 {
     g_return_if_fail(IS_RING_MAIN_WINDOW(self));
     auto priv = RING_MAIN_WINDOW_GET_PRIVATE(RING_MAIN_WINDOW(self));
@@ -216,6 +216,7 @@
                 break;
             case Call::LifeCycleState::PROGRESS:
                 new_call_view = current_call_view_new();
+                g_signal_connect(new_call_view, "video-double-clicked", G_CALLBACK(video_double_clicked), win);
                 current_call_view_set_call_info(CURRENT_CALL_VIEW(new_call_view), idx);
                 /* use the pointer of the call as a unique name */
                 new_call_view_name = g_strdup_printf("%p_current", (void *)CallModel::instance().getCall(idx));
@@ -228,7 +229,6 @@
         gtk_container_remove(GTK_CONTAINER(priv->frame_call), old_call_view);
         gtk_container_add(GTK_CONTAINER(priv->frame_call), new_call_view);
         gtk_widget_show(new_call_view);
-        g_signal_connect(new_call_view, "video-double-clicked", G_CALLBACK(video_double_clicked), win);
         g_free(new_call_view_name);
     } else {
         /* nothing selected in the call model, so show the default screen */