From: Hyongtaek Lim <hyongtaek.lim@samsung.com>
Date: Mon, 27 Jul 2015 23:43:17 +0000 (+0900)
Subject: Add compile option -Werror
X-Git-Tag: submit/tizen/20150728.090711~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6d5530319b4e04a1ae5f68e0ccd580709535d99d;p=platform%2Fcore%2Fmultimedia%2Flibmm-player.git

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
new mode 100644
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 f01b7db..9f12647 100644
--- 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;
 			}