After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 327357 - gst-plugins-base fails to compile with GCC 4.1
gst-plugins-base fails to compile with GCC 4.1
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.6
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-01-17 14:34 UTC by Christian Fredrik Kalager Schaller
Modified: 2006-03-21 15:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed fix (2.05 KB, patch)
2006-01-19 17:52 UTC, Tim-Philipp Müller
none Details | Review
Fix for all warnings in plugins-base with gcc 4.1 (11.14 KB, patch)
2006-03-19 15:35 UTC, Edward Hervey
none Details | Review

Description Christian Fredrik Kalager Schaller 2006-01-17 14:34:17 UTC
cc1: warnings being treated as errors
gstringbuffer.c: In function 'build_linear_format':
gstringbuffer.c:173: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstringbuffer.c:192: warning: dereferencing type-punned pointer will break strict-aliasing rules
Comment 1 Tim-Philipp Müller 2006-01-19 17:52:24 UTC
Created attachment 57664 [details] [review]
proposed fix

Does this fix it?
Comment 2 Christian Fredrik Kalager Schaller 2006-01-23 15:04:51 UTC
With this patch added I think it goes further as I now get:
cc1: warnings being treated as errors
gstcddabasesrc.c: In function 'gst_cdda_base_src_handle_event':
gstcddabasesrc.c:851: warning: value computed is not used
gstcddabasesrc.c:858: warning: value computed is not used
make[4]: *** [libgstcdda_0.10_la-gstcddabasesrc.lo] Error 1
Comment 3 Tim-Philipp Müller 2006-01-23 17:47:25 UTC
Could you do

 % make ERROR_CFLAGS="" 2> warnings.log

and paste the whole bunch of them, so we don't have to do it on a per-file basis? :)

I bet there's a whole bunch more because of that gcc bug with gst_event_ref()/gst_buffer_ref() ...
Comment 4 Christian Fredrik Kalager Schaller 2006-01-23 18:37:21 UTC
gstcddabasesrc.c: In function 'gst_cdda_base_src_handle_event':
gstcddabasesrc.c:851: warning: value computed is not used
gstcddabasesrc.c:858: warning: value computed is not used
gstplaybin.c: In function 'handoff':
gstplaybin.c:375: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstplaybin.c: In function 'gst_play_bin_send_event_to_sink':
gstplaybin.c:940: warning: value computed is not used
gststreamselector.c: In function 'gst_stream_selector_set_property':
gststreamselector.c:202: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstmultifdsink.c: In function 'gst_multi_fd_sink_client_queue_buffer':
gstmultifdsink.c:850: warning: value computed is not used
gstmultifdsink.c: In function 'gst_multi_fd_sink_render':
gstmultifdsink.c:1510: warning: value computed is not used
ximagesink.c: In function 'gst_ximage_buffer_finalize':
ximagesink.c:220: warning: value computed is not used
xvimagesink.c: In function 'gst_xvimage_buffer_finalize':
xvimagesink.c:293: warning: value computed is not used
gstoggdemux.c: In function 'gst_ogg_pad_dispose':
gstoggdemux.c:280: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstoggdemux.c:281: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstoggdemux.c:282: warning: dereferencing type-punned pointer will break strict-aliasing rules
gstoggdemux.c: In function 'gst_ogg_demux_send_event':
gstoggdemux.c:2408: warning: value computed is not used
gstoggmux.c: In function 'gst_ogg_mux_get_headers':
gstoggmux.c:755: warning: value computed is not used
gstoggmux.c: In function 'gst_ogg_mux_send_headers':
gstoggmux.c:871: warning: value computed is not used
gstoggparse.c: In function 'gst_ogg_parse_chain':
gstoggparse.c:437: warning: value computed is not used
gstoggparse.c:473: warning: value computed is not used
gstoggparse.c:489: warning: value computed is not used
seek.c: In function ‘send_event’:
seek.c:1024: warning: value computed is not used
seek.c:1037: warning: value computed is not used
Comment 5 Edward Hervey 2006-03-19 15:35:14 UTC
Created attachment 61545 [details] [review]
Fix for all warnings in plugins-base with gcc 4.1

This one should cover all warnings. Works fine here, including make check.
Comment 6 Edward Hervey 2006-03-21 15:01:57 UTC
Fixed, closing bug.

2006-03-21  Edward Hervey  <edward@fluendo.com>

	* ext/ogg/gstoggdemux.c: (gst_ogg_pad_dispose):
	* gst/playback/gstplaybin.c: (handoff):
	* gst/playback/gststreamselector.c:
	(gst_stream_selector_set_property):
	gcc 4.1 unreferenced pointer fixes.
	* sys/ximage/ximagesink.c: (gst_ximagesink_ximage_put):
	* sys/xvimage/xvimagesink.c: (gst_xvimagesink_xvimage_put):
	gst_buffer_ref() now takes a GstBuffer*.