Add -Wold-style-definition flag

And fix the warnings

diff --git a/configure.ac b/configure.ac
index d6015598fe1f49dbf15e3964c1d4b5c47f35abd4..61b37a55d0fa0345c61421e0d51e1335e34c95af 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -272,10 +272,10 @@ dnl set location of plugin directory
 AG_GST_SET_PLUGINDIR
 
 dnl define an ERROR_CFLAGS Makefile variable
-AG_GST_SET_ERROR_CFLAGS($GST_GIT, [-Wmissing-declarations -Wmissing-prototypes -Wredundant-decls -Wundef -Wwrite-strings])
+AG_GST_SET_ERROR_CFLAGS($GST_GIT, [-Wmissing-declarations -Wmissing-prototypes -Wredundant-decls -Wundef -Wwrite-strings -Wold-style-definition])
 
 dnl define an ERROR_CXXFLAGS Makefile variable
-AG_GST_SET_ERROR_CXXFLAGS($GST_GIT, [-Wmissing-declarations -Wredundant-decls -Wundef -Wwrite-strings])
+AG_GST_SET_ERROR_CXXFLAGS($GST_GIT, [-Wmissing-declarations -Wredundant-decls -Wundef -Wwrite-strings -Wold-style-definition])
 
 dnl define correct level for debugging messages
 AG_GST_SET_LEVEL_DEFAULT($GST_GIT)

diff --git a/ext/annodex/gstcmmlutils.c b/ext/annodex/gstcmmlutils.c
index 6ec682966f11b25ad27ff2e95a77f3bc053a81f0..a66b3f06421bceea86bd61566ff28ae824b9c952 100644 (file)
--- a/ext/annodex/gstcmmlutils.c
+++ b/ext/annodex/gstcmmlutils.c
@@ -228,7 +228,7 @@ overflow:
 
 /* track list */
 GHashTable *
-gst_cmml_track_list_new ()
+gst_cmml_track_list_new (void)
 {
   return g_hash_table_new (g_str_hash, g_str_equal);
 }

diff --git a/ext/wavpack/gstwavpackparse.c b/ext/wavpack/gstwavpackparse.c
index fbb933197f7a54e99bc45dfec73f98d92e6d66d5..47147a891fbe240c7594e1a8bf84b38eaf217097 100644 (file)
--- a/ext/wavpack/gstwavpackparse.c
+++ b/ext/wavpack/gstwavpackparse.c
@@ -56,7 +56,7 @@ GST_DEBUG_CATEGORY_STATIC (gst_wavpack_parse_debug);
 #define GST_CAT_DEFAULT gst_wavpack_parse_debug
 
 static inline GstWavpackParseIndexEntry *
-gst_wavpack_parse_index_entry_new ()
+gst_wavpack_parse_index_entry_new (void)
 {
   return g_slice_new (GstWavpackParseIndexEntry);
 }

diff --git a/gst/effectv/gstwarp.c b/gst/effectv/gstwarp.c
index 0b0c4adcb58d27303a4321cc2b1cbcab29fecc86..72c4eb03d8321a2c6a238d662d056cc100c3ae67 100644 (file)
--- a/gst/effectv/gstwarp.c
+++ b/gst/effectv/gstwarp.c
@@ -115,7 +115,7 @@ gst_warptv_set_caps (GstBaseTransform * btrans, GstCaps * incaps,
 static gint32 sintable[1024 + 256];
 
 static void
-initSinTable ()
+initSinTable (void)
 {
   gint32 *tptr, *tsinptr;
   double i;

diff --git a/gst/rtp/gstrtph263pay.c b/gst/rtp/gstrtph263pay.c
index fd10d9833dcf8c1c3c70b48d0d82cab0b2bae2f9..ea064be65082183b92eb58702f97aa134c773448 100644 (file)
--- a/gst/rtp/gstrtph263pay.c
+++ b/gst/rtp/gstrtph263pay.c
@@ -531,7 +531,7 @@ gst_rtp_h263_pay_get_property (GObject * object, guint prop_id,
 }
 
 static GstRtpH263PayPackage *
-gst_rtp_h263_pay_package_new_empty ()
+gst_rtp_h263_pay_package_new_empty (void)
 {
   return (GstRtpH263PayPackage *) g_malloc0 (sizeof (GstRtpH263PayPackage));
 }

diff --git a/gst/udp/gstmultiudpsink.c b/gst/udp/gstmultiudpsink.c
index 1b8ba65beb80d1acb6323965489410132bce61b4..d27bdbec476f9410fee1f7f66fca47a67623826e 100644 (file)
--- a/gst/udp/gstmultiudpsink.c
+++ b/gst/udp/gstmultiudpsink.c
@@ -377,7 +377,7 @@ gst_multiudpsink_finalize (GObject * object)
 }
 
 static gboolean
-socket_error_is_ignorable ()
+socket_error_is_ignorable (void)
 {
 #ifdef G_OS_WIN32
   /* Windows doesn't seem to have an EAGAIN for sockets */
@@ -388,7 +388,7 @@ socket_error_is_ignorable ()
 }
 
 static int
-socket_last_error_code ()
+socket_last_error_code (void)
 {
 #ifdef G_OS_WIN32
   return WSAGetLastError ();
@@ -398,7 +398,7 @@ socket_last_error_code ()
 }
 
 static gchar *
-socket_last_error_message ()
+socket_last_error_message (void)
 {
 #ifdef G_OS_WIN32
   int errorcode = WSAGetLastError ();

diff --git a/tests/check/elements/cmmldec.c b/tests/check/elements/cmmldec.c
index dff3d20fd8bca5d4111cddc85ce40ddf995f4626..a7d5848ecf331524d4ce2f5541f8a2389d25b7bf 100644 (file)
--- a/tests/check/elements/cmmldec.c
+++ b/tests/check/elements/cmmldec.c
@@ -138,7 +138,7 @@ buffer_unref (void *buffer, void *user_data)
 }
 
 static void
-setup_cmmldec ()
+setup_cmmldec (void)
 {
   GST_DEBUG ("setup_cmmldec");
   cmmldec = gst_check_setup_element ("cmmldec");
@@ -160,7 +160,7 @@ setup_cmmldec ()
 }
 
 static void
-teardown_cmmldec ()
+teardown_cmmldec (void)
 {
   g_list_foreach (buffers, buffer_unref, NULL);
   g_list_free (buffers);
@@ -240,7 +240,7 @@ cmml_tag_message_pop (GstBus * bus, const gchar * tag)
 }
 
 static void
-check_headers ()
+check_headers (void)
 {
   GObject *head_tag;
 
@@ -581,7 +581,7 @@ GST_START_TEST (test_sink_query_convert)
 GST_END_TEST;
 
 static Suite *
-cmmldec_suite ()
+cmmldec_suite (void)
 {
   Suite *s = suite_create ("cmmldec");
 

diff --git a/tests/check/elements/cmmlenc.c b/tests/check/elements/cmmlenc.c
index a39a18c5c6cd018bff79ff58e0b16212c0f593ae..eff12414c859c2c429b15072b368d356fd9de875 100644 (file)
--- a/tests/check/elements/cmmlenc.c
+++ b/tests/check/elements/cmmlenc.c
@@ -152,7 +152,7 @@ buffer_unref (void *buffer, void *user_data)
 }
 
 static void
-setup_cmmlenc ()
+setup_cmmlenc (void)
 {
   guint64 granulerate_n, granulerate_d;
 
@@ -179,7 +179,7 @@ setup_cmmlenc ()
 }
 
 static void
-teardown_cmmlenc ()
+teardown_cmmlenc (void)
 {
   /* free encoded buffers */
   g_list_foreach (buffers, buffer_unref, NULL);
@@ -231,7 +231,7 @@ push_data (const gchar * name, const gchar * data, gint size)
 }
 
 static void
-check_headers ()
+check_headers (void)
 {
   /* push the cmml start tag */
   flow = push_data ("preamble", PREAMBLE, strlen (PREAMBLE));
@@ -473,7 +473,7 @@ GST_START_TEST (test_time_limits)
 GST_END_TEST;
 
 static Suite *
-cmmlenc_suite ()
+cmmlenc_suite (void)
 {
   Suite *s = suite_create ("cmmlenc");
   TCase *tc_general = tcase_create ("general");

diff --git a/tests/check/elements/deinterlace.c b/tests/check/elements/deinterlace.c
index c7c66cf309f8b5af8b636cab1f6384fa84d668da..2f7b29847ce0ff9bb5c9dd3678e0610f736c3f03 100644 (file)
--- a/tests/check/elements/deinterlace.c
+++ b/tests/check/elements/deinterlace.c
@@ -79,7 +79,7 @@ static GstElement *pipeline;
 
 /* sets up deinterlace and shortcut pointers to its pads */
 static void
-setup_deinterlace ()
+setup_deinterlace (void)
 {
   deinterlace = gst_element_factory_make ("deinterlace", NULL);
   fail_unless (deinterlace != NULL);

diff --git a/tests/check/elements/rglimiter.c b/tests/check/elements/rglimiter.c
index 54747d8dc6a90d63a6cd396e2e3256752104cf86..9765794b6b9f141ec46f3a309e035550bb8d7bb0 100644 (file)
--- a/tests/check/elements/rglimiter.c
+++ b/tests/check/elements/rglimiter.c
@@ -101,7 +101,7 @@ static const gfloat test_output[] = {
 };
 
 static GstBuffer *
-create_test_buffer ()
+create_test_buffer (void)
 {
   GstBuffer *buf = gst_buffer_new_and_alloc (sizeof (test_input));
   GstCaps *caps;

diff --git a/tests/check/elements/rtp-payloading.c b/tests/check/elements/rtp-payloading.c
index 1794e9c20d84c6935d7c994a5e9191f599587596..4502fb87c6cfe6ac0344f80663fdf7964d4bb7dd 100644 (file)
--- a/tests/check/elements/rtp-payloading.c
+++ b/tests/check/elements/rtp-payloading.c
@@ -774,7 +774,7 @@ GST_END_TEST;
  * Returns: pointer to the test suite.
  */
 static Suite *
-rtp_payloading_suite ()
+rtp_payloading_suite (void)
 {
   Suite *s = suite_create ("rtp_data_test");
 

diff --git a/tests/check/elements/udpsink.c b/tests/check/elements/udpsink.c
index 62f380dd137a5ee431878db2abbe34fa9d9215c8..10d7df5f9347fdc834f6c039c8fd7ae506669808 100644 (file)
--- a/tests/check/elements/udpsink.c
+++ b/tests/check/elements/udpsink.c
@@ -189,7 +189,7 @@ GST_END_TEST;
  * Returns: pointer to the test suite.
  */
 static Suite *
-udpsink_suite ()
+udpsink_suite (void)
 {
   Suite *s = suite_create ("udpsink_test");
 

diff --git a/tests/check/elements/videofilter.c b/tests/check/elements/videofilter.c
index 4e9b6b1ce683e0f6a1ff53d6cdeff49229c8f265..f7959ec2fcb9c18d3ed28a833de94b80d042525d 100644 (file)
--- a/tests/check/elements/videofilter.c
+++ b/tests/check/elements/videofilter.c
@@ -164,7 +164,7 @@ GST_END_TEST;
 
 
 static Suite *
-videobalance_suite ()
+videobalance_suite (void)
 {
   Suite *s = suite_create ("videobalance");
   TCase *tc_chain = tcase_create ("general");
@@ -176,7 +176,7 @@ videobalance_suite ()
 }
 
 static Suite *
-videoflip_suite ()
+videoflip_suite (void)
 {
   Suite *s = suite_create ("videoflip");
   TCase *tc_chain = tcase_create ("general");
@@ -188,7 +188,7 @@ videoflip_suite ()
 }
 
 static Suite *
-gamma_suite ()
+gamma_suite (void)
 {
   Suite *s = suite_create ("gamma");
   TCase *tc_chain = tcase_create ("general");

diff --git a/tests/check/elements/wavpackdec.c b/tests/check/elements/wavpackdec.c
index 54a764b60fc97062a2be174ff8c537a98d9e16c8..0d1732333646cf4fe5a20cde4c19d181b991c776 100644 (file)
--- a/tests/check/elements/wavpackdec.c
+++ b/tests/check/elements/wavpackdec.c
@@ -67,7 +67,7 @@ static GstStaticPadTemplate srctemplate = GST_STATIC_PAD_TEMPLATE ("src",
     );
 
 static GstElement *
-setup_wavpackdec (dec)
+setup_wavpackdec (void)
 {
   GstElement *wavpackdec;
 

diff --git a/tests/check/generic/states.c b/tests/check/generic/states.c
index 68f807ababf64e41e6d47d25f79c33569a295523..1ed6dc9389916c01218c133a020dab18e8ed6f85 100644 (file)
--- a/tests/check/generic/states.c
+++ b/tests/check/generic/states.c
@@ -31,7 +31,7 @@
 static GList *elements = NULL;
 
 static void
-setup ()
+setup (void)
 {
   GList *features, *f;
   GList *plugins, *p;
@@ -88,7 +88,7 @@ setup ()
 }
 
 static void
-teardown ()
+teardown (void)
 {
   GList *e;
 

diff --git a/tests/icles/v4l2src-test.c b/tests/icles/v4l2src-test.c
index edbfd938888986acdb7883ee103f574091adc171..4fcf48b726dde2b58686cba5d342be9ced815eda 100644 (file)
--- a/tests/icles/v4l2src-test.c
+++ b/tests/icles/v4l2src-test.c
@@ -33,7 +33,7 @@ GMainLoop *loop;
 volatile int exit_read = 0;
 
 static void
-print_options ()
+print_options (void)
 {
   printf ("\nf - to change the fequency\n");
   printf ("i - to change the input\n");