parts of the patch submitted in bug #113913
authorBenjamin Otte <otte@gnome.org>
Fri, 30 Jan 2004 20:48:13 +0000 (20:48 +0000)
committerBenjamin Otte <otte@gnome.org>
Fri, 30 Jan 2004 20:48:13 +0000 (20:48 +0000)
Original commit message from CVS:
2004-01-30  Laurent Vivier <Laurent.Vivier@bull.net>

reviewed by Benjamin Otte
parts of the patch submitted in bug #113913

* configure.ac:
use AC_C_INLINE. Use = instead of == with test
* examples/plugins/example.c:
* gst/autoplug/gstspideridentity.c:
* gst/elements/gstfdsrc.c:
* gst/elements/gstfilesrc.c:
* gst/elements/gstidentity.c:
* gst/elements/gstmultidisksrc.c:
* gst/elements/gststatistics.c:
* gst/gstelement.c:
* gst/gstobject.c:
* gst/gstpad.c:
* gst/gstpipeline.c:
* gst/gstthread.c:
don't end enums with a comma
* gst/gstindex.c: (gst_index_compare_func):
do explicit casting to gint
* gst/gsttrace.c: (gst_trace_text_flush):
#define strsize as a macro

33 files changed:
ChangeLog
configure.ac
docs/gst/tmpl/gstbin.sgml
docs/gst/tmpl/gstclock.sgml
docs/gst/tmpl/gstelement.sgml
docs/gst/tmpl/gstindex.sgml
docs/gst/tmpl/gstobject.sgml
docs/gst/tmpl/gstpadtemplate.sgml
docs/gst/tmpl/gstreamer-unused.sgml
docs/gst/tmpl/gstthread.sgml
docs/gst/tmpl/gstxml.sgml
examples/plugins/example.c
gst/autoplug/gstspideridentity.c
gst/elements/gstfdsrc.c
gst/elements/gstfilesrc.c
gst/elements/gstidentity.c
gst/elements/gstmultidisksrc.c
gst/elements/gstmultifilesrc.c
gst/elements/gststatistics.c
gst/gstelement.c
gst/gstindex.c
gst/gstobject.c
gst/gstpad.c
gst/gstpipeline.c
gst/gstthread.c
gst/gsttrace.c
plugins/elements/gstfdsrc.c
plugins/elements/gstfilesrc.c
plugins/elements/gstidentity.c
plugins/elements/gstmultidisksrc.c
plugins/elements/gstmultifilesrc.c
plugins/elements/gststatistics.c
tests/old/examples/plugins/example.c

index 0019682..27d341d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,28 @@
+2004-01-30  Laurent Vivier <Laurent.Vivier@bull.net>
+
+       reviewed by Benjamin Otte 
+         parts of the patch submitted in bug #113913
+
+       * configure.ac:
+         use AC_C_INLINE. Use = instead of == with test
+       * examples/plugins/example.c:
+       * gst/autoplug/gstspideridentity.c:
+       * gst/elements/gstfdsrc.c:
+       * gst/elements/gstfilesrc.c:
+       * gst/elements/gstidentity.c:
+       * gst/elements/gstmultidisksrc.c:
+       * gst/elements/gststatistics.c:
+       * gst/gstelement.c:
+       * gst/gstobject.c:
+       * gst/gstpad.c:
+       * gst/gstpipeline.c:
+       * gst/gstthread.c:
+         don't end enums with a comma
+       * gst/gstindex.c: (gst_index_compare_func):
+         do explicit casting to gint
+       * gst/gsttrace.c: (gst_trace_text_flush):
+         #define strsize as a macro
+
 2004-01-30  Thomas Vander Stichele  <thomas at apestaart dot org>
 
        * docs/README:
index 4aa8c53..ba85187 100644 (file)
@@ -65,6 +65,7 @@ dnl *NOTE*: dnl this line before release, so release does static too
 dnl AM_DISABLE_STATIC
 
 AC_HEADER_STDC([])
+AC_C_INLINE
 
 dnl define correct errorlevel for debugging messages. We want to have GST_ERROR
 dnl messages printed when running cvs builds
@@ -149,7 +150,7 @@ AC_MSG_RESULT(no)
 
 dnl check for makecontext and define HAVE_MAKECONTEXT if we have it
 AC_CHECK_MCSC()
-if test "$ac_cv_check_mcsc" == "yes"; then
+if test "$ac_cv_check_mcsc" = "yes"; then
   AC_DEFINE_UNQUOTED(HAVE_MAKECONTEXT, $HAVE_MAKECONTEXT,
                      [defined if we have makecontext ()])
 fi
index 68c62da..8a0038e 100644 (file)
@@ -72,6 +72,34 @@ The GstBin object
 </para>
 
 
+<!-- ##### SIGNAL GstBin::element-added ##### -->
+<para>
+
+</para>
+
+@gstbin: the object which received the signal.
+@arg1: the element that was added to the bin
+
+<!-- ##### SIGNAL GstBin::element-removed ##### -->
+<para>
+
+</para>
+
+@gstbin: the object which received the signal.
+@arg1: the element that was removed from the bin
+
+<!-- ##### SIGNAL GstBin::iterate ##### -->
+<para>
+This signal is emitted when a bin iterates, either automatically or
+due to a #gst_bin_iterate() call.  The return value is used to
+determine if the object method handler processed any data.
+In most normal cases, a user-provided signal handler should return
+FALSE.
+</para>
+
+@gstbin: the object which received the signal.
+@Returns: TRUE if the state of the bin was advanced.
+
 <!-- ##### USER_FUNCTION GstBinPrePostIterateFunction ##### -->
 <para>
 The signature of the callback for the post and pre iterate function as set with
@@ -224,31 +252,3 @@ gst_bin_set_pre_iterate_function() and gst_bin_set_post_iterate_function().
 @clock: 
 
 
-<!-- ##### SIGNAL GstBin::element-added ##### -->
-<para>
-
-</para>
-
-@gstbin: the object which received the signal.
-@arg1: the element that was added to the bin
-
-<!-- ##### SIGNAL GstBin::element-removed ##### -->
-<para>
-
-</para>
-
-@gstbin: the object which received the signal.
-@arg1: the element that was removed from the bin
-
-<!-- ##### SIGNAL GstBin::iterate ##### -->
-<para>
-This signal is emitted when a bin iterates, either automatically or
-due to a #gst_bin_iterate() call.  The return value is used to
-determine if the object method handler processed any data.
-In most normal cases, a user-provided signal handler should return
-FALSE.
-</para>
-
-@gstbin: the object which received the signal.
-@Returns: TRUE if the state of the bin was advanced.
-
index 0e9540e..2f67eba 100644 (file)
@@ -235,6 +235,21 @@ The GstClock object
 </para>
 
 
+<!-- ##### ARG GstClock:event-diff ##### -->
+<para>
+
+</para>
+
+<!-- ##### ARG GstClock:max-diff ##### -->
+<para>
+Maximum allowed diff for clock sync requests against the real time.
+</para>
+
+<!-- ##### ARG GstClock:stats ##### -->
+<para>
+Boolean property to activate stat generation on the clock.
+</para>
+
 <!-- ##### FUNCTION gst_clock_set_speed ##### -->
 <para>
 
@@ -402,18 +417,3 @@ The GstClock object
 @id: 
 
 
-<!-- ##### ARG GstClock:event-diff ##### -->
-<para>
-
-</para>
-
-<!-- ##### ARG GstClock:max-diff ##### -->
-<para>
-Maximum allowed diff for clock sync requests against the real time.
-</para>
-
-<!-- ##### ARG GstClock:stats ##### -->
-<para>
-Boolean property to activate stat generation on the clock.
-</para>
-
index 542a16f..9165014 100644 (file)
@@ -71,6 +71,77 @@ The element object
 </para>
 
 
+<!-- ##### SIGNAL GstElement::eos ##### -->
+<para>
+Signal emited when the element goes to PAUSED due to an end-of-stream
+condition.
+</para>
+
+@gstelement: the object which received the signal.
+
+<!-- ##### SIGNAL GstElement::error ##### -->
+<para>
+Is triggered whenever an error occured.
+
+</para>
+
+@gstelement: the object which received the signal.
+@arg1: the error message
+@arg2: 
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@:
+@: 
+
+<!-- ##### SIGNAL GstElement::found-tag ##### -->
+<para>
+
+</para>
+
+@gstelement: the object which received the signal.
+@arg1: 
+@arg2: 
+
+<!-- ##### SIGNAL GstElement::new-pad ##### -->
+<para>
+Is triggered whenever a new pad is added to an element.
+</para>
+
+@gstelement: the object which received the signal.
+@arg1: the new pad that was added
+
+<!-- ##### SIGNAL GstElement::pad-removed ##### -->
+<para>
+Is triggered whenever a pad has been removed from the element.
+</para>
+
+@gstelement: the object which received the signal.
+@arg1: The pad that was removed.
+
+<!-- ##### SIGNAL GstElement::state-change ##### -->
+<para>
+Is triggered whenever the state of an element changes.
+</para>
+
+@gstelement: the object which received the signal.
+@arg1: the new state of the object
+@arg2: 
+
 <!-- ##### MACRO gst_element_get_name ##### -->
 <para>
 Gets the name of the element.
@@ -954,73 +1025,3 @@ Helper macro to create query type functions
 @...: list of query types.
 
 
-<!-- ##### SIGNAL GstElement::eos ##### -->
-<para>
-Signal emited when the element goes to PAUSED due to an end-of-stream
-condition.
-</para>
-
-@gstelement: the object which received the signal.
-
-<!-- ##### SIGNAL GstElement::error ##### -->
-<para>
-Is triggered whenever an error occured.
-
-</para>
-
-@gstelement: the object which received the signal.
-@arg1: the error message
-@arg2: 
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@:
-@: 
-
-<!-- ##### SIGNAL GstElement::found-tag ##### -->
-<para>
-
-</para>
-
-@gstelement: the object which received the signal.
-@arg1: 
-@arg2: 
-
-<!-- ##### SIGNAL GstElement::new-pad ##### -->
-<para>
-Is triggered whenever a new pad is added to an element.
-</para>
-
-@gstelement: the object which received the signal.
-@arg1: the new pad that was added
-
-<!-- ##### SIGNAL GstElement::pad-removed ##### -->
-<para>
-Is triggered whenever a pad has been removed from the element.
-</para>
-
-@gstelement: the object which received the signal.
-@arg1: The pad that was removed.
-
-<!-- ##### SIGNAL GstElement::state-change ##### -->
-<para>
-Is triggered whenever the state of an element changes.
-</para>
-
-@gstelement: the object which received the signal.
-@arg1: the new state of the object
-@arg2: 
-
index d828fa3..cdc3cfa 100644 (file)
@@ -209,6 +209,19 @@ The GstIndex object
 </para>
 
 
+<!-- ##### SIGNAL GstIndex::entry-added ##### -->
+<para>
+Is emited when a new entry is added to the index.
+</para>
+
+@gstindex: the object which received the signal.
+@arg1: The entry added to the index.
+
+<!-- ##### ARG GstIndex:resolver ##### -->
+<para>
+
+</para>
+
 <!-- ##### FUNCTION gst_index_new ##### -->
 <para>
 
@@ -401,16 +414,3 @@ The GstIndex object
 @id: 
 
 
-<!-- ##### SIGNAL GstIndex::entry-added ##### -->
-<para>
-Is emited when a new entry is added to the index.
-</para>
-
-@gstindex: the object which received the signal.
-@arg1: The entry added to the index.
-
-<!-- ##### ARG GstIndex:resolver ##### -->
-<para>
-
-</para>
-
index 52616a2..cf69ab9 100644 (file)
@@ -40,6 +40,47 @@ The GstObject
 </para>
 
 
+<!-- ##### SIGNAL GstObject::deep-notify ##### -->
+<para>
+The deep notify signal is used to be notified of property changes.
+it is typically attached to the toplevel bin to receive notifications
+from all the elements contained in that bin.
+</para>
+
+@gstobject: the object which received the signal.
+@arg1: the object that originated the signal
+@arg2: the property that changed
+
+<!-- ##### SIGNAL GstObject::object-saved ##### -->
+<para>
+Is trigered whenever a new object is saved to XML. You can connect to
+this signal to insert custom XML tags into the core XML.
+</para>
+
+@gstobject: the object which received the signal.
+@arg1: the xmlNodePtr of the parent node
+
+<!-- ##### SIGNAL GstObject::parent-set ##### -->
+<para>
+Is emitted when the parent of an object is set.
+</para>
+
+@gstobject: the object which received the signal.
+@arg1: the new parent
+
+<!-- ##### SIGNAL GstObject::parent-unset ##### -->
+<para>
+Is emitted when the parent of an object is unset.
+</para>
+
+@gstobject: the object which received the signal.
+@arg1: the old parent
+
+<!-- ##### ARG GstObject:name ##### -->
+<para>
+The name of the object
+</para>
+
 <!-- ##### MACRO GST_FLAGS ##### -->
 <para>
 This macro returns the entire set of flags for the object.
@@ -290,44 +331,3 @@ Check if the object has been destroyed.
 @Returns: 
 
 
-<!-- ##### SIGNAL GstObject::deep-notify ##### -->
-<para>
-The deep notify signal is used to be notified of property changes.
-it is typically attached to the toplevel bin to receive notifications
-from all the elements contained in that bin.
-</para>
-
-@gstobject: the object which received the signal.
-@arg1: the object that originated the signal
-@arg2: the property that changed
-
-<!-- ##### SIGNAL GstObject::object-saved ##### -->
-<para>
-Is trigered whenever a new object is saved to XML. You can connect to
-this signal to insert custom XML tags into the core XML.
-</para>
-
-@gstobject: the object which received the signal.
-@arg1: the xmlNodePtr of the parent node
-
-<!-- ##### SIGNAL GstObject::parent-set ##### -->
-<para>
-Is emitted when the parent of an object is set.
-</para>
-
-@gstobject: the object which received the signal.
-@arg1: the new parent
-
-<!-- ##### SIGNAL GstObject::parent-unset ##### -->
-<para>
-Is emitted when the parent of an object is unset.
-</para>
-
-@gstobject: the object which received the signal.
-@arg1: the old parent
-
-<!-- ##### ARG GstObject:name ##### -->
-<para>
-The name of the object
-</para>
-
index a273692..b20d655 100644 (file)
@@ -87,6 +87,15 @@ The padtemplate object.
 </para>
 
 
+<!-- ##### SIGNAL GstPadTemplate::pad-created ##### -->
+<para>
+This signal is fired when an element creates a pad from this 
+template.
+</para>
+
+@gstpadtemplate: the object which received the signal.
+@arg1: The pad that was created.
+
 <!-- ##### ENUM GstPadTemplateFlags ##### -->
 <para>
 Flags for the padtemplate
@@ -168,12 +177,3 @@ Check if the properties of the padtemplate are fixed
 @Returns: 
 
 
-<!-- ##### SIGNAL GstPadTemplate::pad-created ##### -->
-<para>
-This signal is fired when an element creates a pad from this 
-template.
-</para>
-
-@gstpadtemplate: the object which received the signal.
-@arg1: The pad that was created.
-
index ee35960..3015f33 100644 (file)
@@ -6033,6 +6033,15 @@ Query the element for the current mime type
 </para>
 
 
+<!-- ##### SIGNAL GstXML::object-loaded ##### -->
+<para>
+
+</para>
+
+@gstxml: the object which received the signal.
+@arg1: 
+@arg2: 
+
 <!-- ##### USER_FUNCTION GstXMLRegistryAddPathList ##### -->
 <para>
 
index 6d70219..38a7691 100644 (file)
@@ -34,15 +34,6 @@ The GstThread object
 </para>
 
 
-<!-- ##### FUNCTION gst_thread_new ##### -->
-<para>
-
-</para>
-
-@name: 
-@Returns: 
-
-
 <!-- ##### SIGNAL GstThread::shutdown ##### -->
 <para>
 
@@ -55,3 +46,12 @@ The GstThread object
 The thread priority
 </para>
 
+<!-- ##### FUNCTION gst_thread_new ##### -->
+<para>
+
+</para>
+
+@name: 
+@Returns: 
+
+
index 08450b2..ea4e4bd 100644 (file)
@@ -105,25 +105,3 @@ All GstElements can be serialized to an XML presentation and subsequently loaded
 @Returns: 
 
 
-<!-- ##### SIGNAL GstXML::object-loaded ##### -->
-<para>
-
-</para>
-
-@: 
-@: 
-@:
-
-@gstxml: the object which received the signal.
-@arg1: 
-@arg2: 
-
-<!-- ##### SIGNAL GstXML::object-loaded ##### -->
-<para>
-
-</para>
-
-@gstxml: the object which received the signal.
-@arg1: 
-@arg2: 
-
index ad47dcd..796d339 100644 (file)
@@ -49,7 +49,7 @@ enum {
  */
 enum {
   ARG_0,
-  ARG_ACTIVE,
+  ARG_ACTIVE
   /* FILL ME */
 };
 
index 7cdbbfa..24c8f60 100644 (file)
@@ -64,7 +64,7 @@ enum {
 };
 
 enum {
-  ARG_0,
+  ARG_0
   /* FILL ME */
 };
 
index d45ff18..ffabc94 100644 (file)
@@ -55,7 +55,7 @@ enum {
 enum {
   ARG_0,
   ARG_FD,
-  ARG_BLOCKSIZE,
+  ARG_BLOCKSIZE
 };
 
 #define _do_init(bla) \
index dab1fd4..c47a7ff 100644 (file)
@@ -99,7 +99,7 @@ enum {
   ARG_FD,
   ARG_BLOCKSIZE,
   ARG_MMAPSIZE,
-  ARG_TOUCH,
+  ARG_TOUCH
 };
 
 GST_PAD_EVENT_MASK_FUNCTION (gst_filesrc_get_event_mask,
index e369cdb..c6b9c36 100644 (file)
@@ -57,7 +57,7 @@ enum {
   ARG_DROP_PROBABILITY,
   ARG_SILENT,
   ARG_LAST_MESSAGE,
-  ARG_DUMP,
+  ARG_DUMP
 };
 
 
index eef3a42..6f3b304 100644 (file)
@@ -55,7 +55,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_LOCATIONS,
+  ARG_LOCATIONS
 };
 
 #define _do_init(bla) \
index eef3a42..6f3b304 100644 (file)
@@ -55,7 +55,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_LOCATIONS,
+  ARG_LOCATIONS
 };
 
 #define _do_init(bla) \
index 688727c..65e578b 100644 (file)
@@ -54,7 +54,7 @@ enum {
   ARG_EVENT_UPDATE_FREQ,
   ARG_UPDATE_ON_EOS,
   ARG_UPDATE,
-  ARG_SILENT,
+  ARG_SILENT
 };
 
 
index 0788407..24cc8d1 100644 (file)
@@ -47,7 +47,7 @@ enum {
 };
 
 enum {
-  ARG_0,
+  ARG_0
   /* FILL ME */
 };
 
index 23f6ce7..9a5951f 100644 (file)
@@ -35,7 +35,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_RESOLVER,
+  ARG_RESOLVER
   /* FILL ME */
 };
 
@@ -710,7 +710,7 @@ gst_index_compare_func (gconstpointer a,
                         gconstpointer b,
                         gpointer user_data)
 {
-  return a - b;  
+  return (gint)a - (gint)b;  
 }
 
 /**
index d6ea419..2fd1748 100644 (file)
@@ -42,7 +42,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_NAME,
+  ARG_NAME
   /* FILL ME */
 };
 
index b8f884e..2f10022 100644 (file)
@@ -126,7 +126,7 @@ enum {
 enum {
   REAL_ARG_0,
   REAL_ARG_CAPS,
-  REAL_ARG_ACTIVE,
+  REAL_ARG_ACTIVE
   /* FILL ME */
 };
 
index aa55d91..d7b4166 100644 (file)
@@ -40,7 +40,7 @@ enum {
 };
 
 enum {
-  ARG_0,
+  ARG_0
   /* FILL ME */
 };
 
index b28d892..97ced94 100644 (file)
@@ -54,7 +54,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_PRIORITY,
+  ARG_PRIORITY
 };
 
 
index 6130e7d..0c9ed79 100644 (file)
@@ -108,8 +108,8 @@ void
 gst_trace_text_flush (GstTrace * trace)
 {
   int i;
-  const int strsize = 20 + 1 + 10 + 1 + 10 + 1 + 112 + 1 + 1;
-  char str[strsize];
+#define STRSIZE (20 + 1 + 10 + 1 + 10 + 1 + 112 + 1 + 1)
+  char str[STRSIZE];
 
   if (!trace) {
     trace = _gst_trace_default;
@@ -118,12 +118,13 @@ gst_trace_text_flush (GstTrace * trace)
   }
 
   for (i = 0; i < trace->bufoffset; i++) {
-    snprintf (str, strsize, "%20" G_GINT64_FORMAT " %10d %10d %s\n",
+    snprintf (str, STRSIZE, "%20" G_GINT64_FORMAT " %10d %10d %s\n",
              trace->buf[i].timestamp,
              trace->buf[i].sequence, trace->buf[i].data, trace->buf[i].message);
     write (trace->fd, str, strlen (str));
   }
   trace->bufoffset = 0;
+#undef STRSIZE
 }
 
 void
index d45ff18..ffabc94 100644 (file)
@@ -55,7 +55,7 @@ enum {
 enum {
   ARG_0,
   ARG_FD,
-  ARG_BLOCKSIZE,
+  ARG_BLOCKSIZE
 };
 
 #define _do_init(bla) \
index dab1fd4..c47a7ff 100644 (file)
@@ -99,7 +99,7 @@ enum {
   ARG_FD,
   ARG_BLOCKSIZE,
   ARG_MMAPSIZE,
-  ARG_TOUCH,
+  ARG_TOUCH
 };
 
 GST_PAD_EVENT_MASK_FUNCTION (gst_filesrc_get_event_mask,
index e369cdb..c6b9c36 100644 (file)
@@ -57,7 +57,7 @@ enum {
   ARG_DROP_PROBABILITY,
   ARG_SILENT,
   ARG_LAST_MESSAGE,
-  ARG_DUMP,
+  ARG_DUMP
 };
 
 
index eef3a42..6f3b304 100644 (file)
@@ -55,7 +55,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_LOCATIONS,
+  ARG_LOCATIONS
 };
 
 #define _do_init(bla) \
index eef3a42..6f3b304 100644 (file)
@@ -55,7 +55,7 @@ enum {
 
 enum {
   ARG_0,
-  ARG_LOCATIONS,
+  ARG_LOCATIONS
 };
 
 #define _do_init(bla) \
index 688727c..65e578b 100644 (file)
@@ -54,7 +54,7 @@ enum {
   ARG_EVENT_UPDATE_FREQ,
   ARG_UPDATE_ON_EOS,
   ARG_UPDATE,
-  ARG_SILENT,
+  ARG_SILENT
 };
 
 
index ad47dcd..796d339 100644 (file)
@@ -49,7 +49,7 @@ enum {
  */
 enum {
   ARG_0,
-  ARG_ACTIVE,
+  ARG_ACTIVE
   /* FILL ME */
 };