Add compile option -Werror

Signed-off-by: Hyongtaek Lim <hyongtaek.lim@samsung.com>
Change-Id: I0147c6273a99b895dafbf332a9fb1e1fc39e95f3

diff --git a/src/Makefile.am b/src/Makefile.am
old mode 100755 (executable)
new mode 100644 (file)
index 22a2cf7..16561ec
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -32,7 +32,7 @@ libmmfplayer_la_CFLAGS =  -I$(srcdir)/include \
                        $(ICU_CFLAGS) \
                        $(UTILX_CFLAGS) \
                        $(MEDIAPACKET_CFLAGS) \
-                       -Wno-deprecated -Wno-deprecated-declarations
+                       -Werror -Wno-deprecated -Wno-deprecated-declarations -Wno-cpp
 
 noinst_HEADERS = include/mm_player_utils.h \
                 include/mm_player_ini.h \

diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index f01b7db6e1807d4616c31797e1052cdfe39ef99d..9f12647a0c392ca30dde5ed05ba160274e2b9d7b 100644 (file)
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -804,7 +804,7 @@ __mmplayer_device_change_trigger_post_process(mm_player_t* player)
 
        player->post_proc.id = g_idle_add((GSourceFunc)__mmplayer_device_change_post_process, (gpointer)player);
 }
-
+#if 0
 /* NOTE : Sound module has different latency according to output device So,
  * synchronization problem can be happened whenever device is changed.
  * To avoid this issue, we do reset avsystem or seek as workaroud.
@@ -851,7 +851,7 @@ __mmplayer_sound_device_info_changed_cb_func (MMSoundDevice_t device_h, int chan
 
        __mmplayer_device_change_trigger_post_process(player);
 }
-
+#endif
 /* This function should be called after the pipeline goes PAUSED or higher
 state. */
 gboolean
@@ -4325,7 +4325,7 @@ __mmplayer_gst_decode_callback(GstElement *elem, GstPad *pad, gpointer data) //
                        /* run */
                        if (GST_STATE_CHANGE_FAILURE == gst_element_set_state (sinkbin, GST_STATE_PAUSED))
                        {
-                               debug_error("failed to set state(PLAYING) to sinkbin\n");
+                               debug_error("failed to set state(PAUSED) to sinkbin\n");
                                goto ERROR;
                        }
 
@@ -4333,7 +4333,7 @@ __mmplayer_gst_decode_callback(GstElement *elem, GstPad *pad, gpointer data) //
                        {
                          if (GST_STATE_CHANGE_FAILURE == gst_element_set_state (text_selector, GST_STATE_PAUSED))
                          {
-                           debug_error("failed to set state(READY) to sinkbin\n");
+                           debug_error("failed to set state(PAUSED) to sinkbin\n");
                            goto ERROR;
                          }
                        }
@@ -4686,17 +4686,15 @@ _mmplayer_update_video_param(mm_player_t* player) // @
                        if ( surface )
                        {
 #ifdef HAVE_WAYLAND
-                               int wl_surface = 0;
-                               wl_surface = (int*)surface;
-                               debug_log("set video param : xid %p", (int*)surface);
-                               if (wl_surface)
-                               {
-                                       gst_video_overlay_set_window_handle( GST_VIDEO_OVERLAY( player->pipeline->videobin[MMPLAYER_V_SINK].gst ), (int*)surface );
-                                       /* After setting window handle, set render      rectangle */
-                                       gst_video_overlay_set_render_rectangle(
-                                                GST_VIDEO_OVERLAY( player->pipeline->videobin[MMPLAYER_V_SINK].gst ),
-                                                wl_window_x,wl_window_y,wl_window_width,wl_window_height);
-                               }
+                               guintptr wl_surface = (guintptr)surface;
+                               debug_log("set video param : wayland surface %p", surface);
+                               gst_video_overlay_set_window_handle(
+                                               GST_VIDEO_OVERLAY( player->pipeline->videobin[MMPLAYER_V_SINK].gst ),
+                                               wl_surface );
+                               /* After setting window handle, set render      rectangle */
+                               gst_video_overlay_set_render_rectangle(
+                                        GST_VIDEO_OVERLAY( player->pipeline->videobin[MMPLAYER_V_SINK].gst ),
+                                        wl_window_x,wl_window_y,wl_window_width,wl_window_height);
 #else // HAVE_X11
                                int xwin_id = 0;
                                xwin_id = *(int*)surface;
@@ -11734,7 +11732,7 @@ __mmplayer_try_to_plug(mm_player_t* player, GstPad *pad, const GstCaps *caps) //
                        /* running */
                        if ( GST_STATE_CHANGE_FAILURE == gst_element_set_state(queue, GST_STATE_PAUSED) )
                        {
-                               debug_warning("failed to set state READY to queue\n");
+                               debug_warning("failed to set state PAUSED to queue\n");
                                goto ERROR;
                        }