More Michael fixage

Original commit message from CVS:
More Michael fixage

ChangeLog

+2001-12-13  Michael Meeks  <michael@ximian.com>
+
+	* gst/elements/*.c: (..._class_init): clean to use
+	gst_populate_std_props.
+
+	* gst/gstelement.c
+	(gst_element_populate_std_props): impl, simply for now,
+	with a few standard properties.
+	(gst_element_install_std_props): impl nice helper.
+
 2001-12-13  Michael Meeks  <michael@ximian.com>
 
 	* gst/gstplugin.c (_gst_plugin_initialize): g_error
@@ -55,33 +65,6 @@
 
 	* configure.ac: update libxml checks.
 
-2001-12-13  Michael Meeks  <michael@ximian.com>
-
-	* plugins/xvideosink/Makefile.am: add $(X_LIBS)
-
-	* examples/typefind/typefind.c (type_found): upd.
-	xml code.
-
-	* plugins/mikmod/gstmikmod.c
-	(gst_mikmod_set_property): same memory bug here -
-	unfixed.
-
-	* plugins/oss/gstosssrc.c
-	(gst_osssrc_set_property): fix memory issue.
-	(gst_osssrc_init): dup the string - argh; quit here -
-	such code should be stuck in a common base class not
-	cut and pasted.
-
-	* plugins/oss/gstosssink.c
-	(gst_osssink_init): manage the device allocation.
-	(gst_osssink_finalize): impl.
-	(gst_osssink_class_init): upd.
-	(gst_osssink_set_property): upd.
-
-	* acconfig.h: cover HAVE_LIBXML2
-
-	* configure.ac: update libxml checks.
-
 2001-12-11 22:10  thomasvs
 
 	* lots of build fixes for 0.3.0 release

gst/elements/gstdisksink.c

@@ -98,9 +98,10 @@ gst_disksink_class_init (GstDiskSinkClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL,G_PARAM_READWRITE)); // CHECKME!
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location", ARG_LOCATION, G_PARAM_READWRITE,
+	  NULL);
 
   gst_disksink_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

gst/elements/gstdisksrc.c

@@ -108,18 +108,13 @@ gst_disksrc_class_init (GstDiskSrcClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL,G_PARAM_READWRITE)); // CHECKME!
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_long("offset","offset","offset",
-                     G_MINLONG,G_MAXLONG,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FILESIZE,
-    g_param_spec_long("filesize","filesize","filesize",
-                     G_MINLONG,G_MAXLONG,0,G_PARAM_READABLE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READWRITE,
+	  "filesize",     ARG_FILESIZE,     G_PARAM_READABLE,
+	  NULL);
 
   gobject_class->set_property = gst_disksrc_set_property;
   gobject_class->get_property = gst_disksrc_get_property;

gst/elements/gstfakesink.c

@@ -107,12 +107,12 @@ gst_fakesink_class_init (GstFakeSinkClass *klass)
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_NUM_SINKS,
     g_param_spec_int ("num_sinks", "num_sinks", "num_sinks",
                       1, G_MAXINT, 1, G_PARAM_READABLE)); 
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SILENT,
-    g_param_spec_boolean ("silent", "silent", "silent",
-                          FALSE, G_PARAM_READWRITE)); 
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DUMP,
-    g_param_spec_boolean ("dump", "dump", "dump",
-                          FALSE, G_PARAM_READWRITE)); 
+
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "silent", ARG_SILENT, G_PARAM_READWRITE,
+	  "dump",   ARG_DUMP,   G_PARAM_READWRITE,
+	  NULL);
 
   gst_fakesink_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

gst/elements/gstfakesrc.c

@@ -231,12 +231,12 @@ gst_fakesrc_class_init (GstFakeSrcClass *klass)
   g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_EOS,
     g_param_spec_boolean("eos","eos","eos",
                          TRUE,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_SILENT,
-    g_param_spec_boolean("silent","silent","silent",
-                         FALSE, G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DUMP,
-    g_param_spec_boolean ("dump","dump","dump",
-                          FALSE, G_PARAM_READWRITE)); 
+
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "silent", ARG_SILENT, G_PARAM_READWRITE,
+	  "dump",   ARG_DUMP,   G_PARAM_READWRITE,
+	  NULL);
 
   gst_fakesrc_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

gst/elements/gstfdsink.c

@@ -89,9 +89,10 @@ gst_fdsink_class_init (GstFdSinkClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FD,
-    g_param_spec_int("fd","fd","fd",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "fd", ARG_FD, G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_fdsink_set_property;
   gobject_class->get_property = gst_fdsink_get_property;

gst/elements/gstfdsrc.c

@@ -98,15 +98,12 @@ gst_fdsrc_class_init (GstFdSrcClass *klass)
 
   parent_class = g_type_class_ref(GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        "",G_PARAM_WRITABLE)); // CHECKME!
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_int("offset","offset","offset",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_WRITABLE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READABLE,
+	  NULL);
 
   gobject_class->set_property = gst_fdsrc_set_property;
   gobject_class->get_property = gst_fdsrc_get_property;

gst/elements/gstfilesrc.c

@@ -146,27 +146,16 @@ gst_filesrc_class_init (GstFileSrcClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","File Location","Location of the file to read",
-                        NULL,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FILESIZE,
-    g_param_spec_int64("filesize","File Size","Size of the file being read",
-                       0,G_MAXINT64,0,G_PARAM_READABLE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FD,
-    g_param_spec_int("fd","File-descriptor","File-descriptor for the file being read",
-                     0,G_MAXINT,0,G_PARAM_READABLE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BLOCKSIZE,
-    g_param_spec_ulong("blocksize","Block Size","Block size to read per buffer",
-                       0,G_MAXULONG,4096,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_int64("offset","File Offset","Byte offset of current read pointer",
-                       0,G_MAXINT64,0,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_MAPSIZE,
-    g_param_spec_ulong("mmapsize","mmap() Block Size","Size in bytes of mmap()d regions",
-                       0,G_MAXULONG,4*1048576,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_TOUCH,
-    g_param_spec_boolean("touch","Touch read data","Touch data to force disk read before push()",
-                         TRUE,G_PARAM_READWRITE));
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "fd",           ARG_FD,           G_PARAM_READABLE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READWRITE,
+	  "filesize",     ARG_FILESIZE,     G_PARAM_READABLE,
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "blocksize",    ARG_BLOCKSIZE,    G_PARAM_READWRITE,
+	  "mmapsize",     ARG_MAPSIZE,      G_PARAM_READWRITE,
+	  "touch",        ARG_TOUCH,        G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_filesrc_set_property;
   gobject_class->get_property = gst_filesrc_get_property;

gst/elements/gsthttpsrc.c

@@ -102,12 +102,11 @@ gst_httpsrc_class_init (GstHttpSrcClass *klass)
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL, G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_httpsrc_set_property;
   gobject_class->get_property = gst_httpsrc_get_property;

gst/gstelement.c

@@ -22,6 +22,7 @@
 
 //#define GST_DEBUG_ENABLED
 #include <glib.h>
+#include <stdarg.h>
 #include "gst_private.h"
 
 #include "gstelement.h"
@@ -1300,3 +1301,136 @@ gst_element_statename (GstElementState state)
   }
   return "";
 }
+
+static void
+gst_element_populate_std_props (GObjectClass *klass,
+				const char   *prop_name,
+				guint         arg_id,
+				GParamFlags   flags)
+{
+	GQuark      prop_id = g_quark_from_string (prop_name);
+	GParamSpec *pspec;
+
+	static GQuark fd_id = 0;
+	static GQuark blocksize_id;
+	static GQuark bytesperread_id;
+	static GQuark dump_id;
+	static GQuark filesize_id;
+	static GQuark mmapsize_id;
+	static GQuark location_id;
+	static GQuark offset_id;
+	static GQuark silent_id;
+	static GQuark touch_id;
+	
+	if (!fd_id) {
+		fd_id           = g_quark_from_static_string ("fd");
+		blocksize_id    = g_quark_from_static_string ("blocksize");
+		bytesperread_id = g_quark_from_static_string ("bytesperread");
+		dump_id         = g_quark_from_static_string ("dump");
+		filesize_id     = g_quark_from_static_string ("filesize");
+		mmapsize_id     = g_quark_from_static_string ("mmapsize");
+		location_id     = g_quark_from_static_string ("location");
+		offset_id       = g_quark_from_static_string ("offset");
+		silent_id       = g_quark_from_static_string ("silent");
+		touch_id        = g_quark_from_static_string ("touch");
+	}
+
+	if (prop_id == fd_id) {
+		g_param_spec_int (
+			"fd", "File-descriptor",
+			"File-descriptor for the file being read",
+			0, G_MAXINT, 0, flags);
+
+	} else if (prop_id == blocksize_id) {
+		pspec = g_param_spec_ulong (
+			"blocksize", "Block Size",
+			"Block size to read per buffer",
+			0, G_MAXULONG, 4096, flags);
+		
+	} else if (prop_id == bytesperread_id) {
+		pspec = g_param_spec_int (
+			"bytesperread", "bytesperread", "bytesperread",
+			G_MININT, G_MAXINT, 0, flags);
+
+	} else if (prop_id == dump_id) {
+		pspec = g_param_spec_boolean (
+			"dump", "dump", "dump",
+			FALSE, flags);
+
+	} else if (prop_id == filesize_id) {
+		pspec = g_param_spec_int64 (
+			"filesize", "File Size",
+			"Size of the file being read",
+			0, G_MAXINT64, 0, flags);
+		
+	} else if (prop_id == mmapsize_id) {
+		pspec = g_param_spec_ulong (
+			"mmapsize", "mmap() Block Size",
+			"Size in bytes of mmap()d regions",
+			0, G_MAXULONG, 4 * 1048576, flags);
+		
+	} else if (prop_id == location_id) {
+		pspec = g_param_spec_string (
+			"location", "File Location",
+			"Location of the file to read",
+			NULL, flags);
+		
+	} else if (prop_id == offset_id) {
+		pspec = g_param_spec_int64 (
+			"offset", "File Offset",
+			"Byte offset of current read pointer",
+			0, G_MAXINT64, 0, flags);
+
+	} else if (prop_id == silent_id) {
+		pspec = g_param_spec_boolean (
+			"silent", "silent", "silent",
+			FALSE, flags);
+
+	} else if (prop_id == touch_id) {
+		pspec = g_param_spec_boolean (
+			"touch", "Touch read data",
+			"Touch data to force disk read before push ()",
+			TRUE, flags);
+	}	
+
+	g_object_class_install_property (klass, arg_id, pspec);
+}
+
+/**
+ * gst_element_install_std_props:
+ * @klass: the class to add the properties to
+ * @first_name: the first in a NULL terminated
+ * 'name', 'id', 'flags' triplet list.
+ * 
+ * Add a list of standardized properties with types to the @klass.
+ * the id is for the property switch in your get_prop method, and
+ * the flags determine readability / writeability.
+ **/
+void
+gst_element_install_std_props (GstElementClass *klass,
+			       const char      *first_name,
+			       ...)
+{
+	const char *name;
+
+	va_list args;
+
+	g_return_if_fail (GST_IS_ELEMENT_CLASS (klass));
+
+	va_start (args, first_name);
+
+	name = first_name;
+
+	while (name) {
+		int arg_id = va_arg (args, int);
+		int flags  = va_arg (args, int);
+
+		gst_element_populate_std_props (
+			(GObjectClass *) klass, name, arg_id, flags);
+		
+		name = va_arg (args, char *);
+	}
+
+	va_end (args);
+}
+

gst/gstelement.h

@@ -225,6 +225,10 @@ void			gst_element_error		(GstElement *element, const gchar *error);
 
 GstElementFactory*	gst_element_get_factory		(GstElement *element);
 
+void                    gst_element_install_std_props   (GstElementClass *klass,
+							 const char      *first_name, ...);
+
+
 #ifndef GST_DISABLE_LOADSAVE
 /* XML write and read */
 GstElement*		gst_element_restore_thyself	(xmlNodePtr self, GstObject *parent);
@@ -293,7 +297,6 @@ GstElement*		gst_elementfactory_create		(GstElementFactory *factory,
 /* FIXME this name is wrong, probably so is the one above it */
 GstElement*		gst_elementfactory_make			(const gchar *factoryname, const gchar *name);
 
-
 #ifdef __cplusplus
 }
 #endif /* __cplusplus */

plugins/elements/gstdisksink.c

@@ -98,9 +98,10 @@ gst_disksink_class_init (GstDiskSinkClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL,G_PARAM_READWRITE)); // CHECKME!
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location", ARG_LOCATION, G_PARAM_READWRITE,
+	  NULL);
 
   gst_disksink_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

plugins/elements/gstdisksrc.c

@@ -108,18 +108,13 @@ gst_disksrc_class_init (GstDiskSrcClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL,G_PARAM_READWRITE)); // CHECKME!
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_long("offset","offset","offset",
-                     G_MINLONG,G_MAXLONG,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FILESIZE,
-    g_param_spec_long("filesize","filesize","filesize",
-                     G_MINLONG,G_MAXLONG,0,G_PARAM_READABLE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READWRITE,
+	  "filesize",     ARG_FILESIZE,     G_PARAM_READABLE,
+	  NULL);
 
   gobject_class->set_property = gst_disksrc_set_property;
   gobject_class->get_property = gst_disksrc_get_property;

plugins/elements/gstfakesink.c

@@ -107,12 +107,12 @@ gst_fakesink_class_init (GstFakeSinkClass *klass)
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_NUM_SINKS,
     g_param_spec_int ("num_sinks", "num_sinks", "num_sinks",
                       1, G_MAXINT, 1, G_PARAM_READABLE)); 
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SILENT,
-    g_param_spec_boolean ("silent", "silent", "silent",
-                          FALSE, G_PARAM_READWRITE)); 
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DUMP,
-    g_param_spec_boolean ("dump", "dump", "dump",
-                          FALSE, G_PARAM_READWRITE)); 
+
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "silent", ARG_SILENT, G_PARAM_READWRITE,
+	  "dump",   ARG_DUMP,   G_PARAM_READWRITE,
+	  NULL);
 
   gst_fakesink_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

plugins/elements/gstfakesrc.c

@@ -231,12 +231,12 @@ gst_fakesrc_class_init (GstFakeSrcClass *klass)
   g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_EOS,
     g_param_spec_boolean("eos","eos","eos",
                          TRUE,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_SILENT,
-    g_param_spec_boolean("silent","silent","silent",
-                         FALSE, G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DUMP,
-    g_param_spec_boolean ("dump","dump","dump",
-                          FALSE, G_PARAM_READWRITE)); 
+
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "silent", ARG_SILENT, G_PARAM_READWRITE,
+	  "dump",   ARG_DUMP,   G_PARAM_READWRITE,
+	  NULL);
 
   gst_fakesrc_signals[SIGNAL_HANDOFF] =
     g_signal_new ("handoff", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,

plugins/elements/gstfdsink.c

@@ -89,9 +89,10 @@ gst_fdsink_class_init (GstFdSinkClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FD,
-    g_param_spec_int("fd","fd","fd",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "fd", ARG_FD, G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_fdsink_set_property;
   gobject_class->get_property = gst_fdsink_get_property;

plugins/elements/gstfdsrc.c

@@ -98,15 +98,12 @@ gst_fdsrc_class_init (GstFdSrcClass *klass)
 
   parent_class = g_type_class_ref(GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        "",G_PARAM_WRITABLE)); // CHECKME!
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_int("offset","offset","offset",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_WRITABLE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READABLE,
+	  NULL);
 
   gobject_class->set_property = gst_fdsrc_set_property;
   gobject_class->get_property = gst_fdsrc_get_property;

plugins/elements/gstfilesrc.c

@@ -146,27 +146,16 @@ gst_filesrc_class_init (GstFileSrcClass *klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","File Location","Location of the file to read",
-                        NULL,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FILESIZE,
-    g_param_spec_int64("filesize","File Size","Size of the file being read",
-                       0,G_MAXINT64,0,G_PARAM_READABLE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FD,
-    g_param_spec_int("fd","File-descriptor","File-descriptor for the file being read",
-                     0,G_MAXINT,0,G_PARAM_READABLE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BLOCKSIZE,
-    g_param_spec_ulong("blocksize","Block Size","Block size to read per buffer",
-                       0,G_MAXULONG,4096,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_OFFSET,
-    g_param_spec_int64("offset","File Offset","Byte offset of current read pointer",
-                       0,G_MAXINT64,0,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_MAPSIZE,
-    g_param_spec_ulong("mmapsize","mmap() Block Size","Size in bytes of mmap()d regions",
-                       0,G_MAXULONG,4*1048576,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_TOUCH,
-    g_param_spec_boolean("touch","Touch read data","Touch data to force disk read before push()",
-                         TRUE,G_PARAM_READWRITE));
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "fd",           ARG_FD,           G_PARAM_READABLE,
+	  "offset",       ARG_OFFSET,       G_PARAM_READWRITE,
+	  "filesize",     ARG_FILESIZE,     G_PARAM_READABLE,
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "blocksize",    ARG_BLOCKSIZE,    G_PARAM_READWRITE,
+	  "mmapsize",     ARG_MAPSIZE,      G_PARAM_READWRITE,
+	  "touch",        ARG_TOUCH,        G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_filesrc_set_property;
   gobject_class->get_property = gst_filesrc_get_property;

plugins/elements/gsthttpsrc.c

@@ -102,12 +102,11 @@ gst_httpsrc_class_init (GstHttpSrcClass *klass)
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_LOCATION,
-    g_param_spec_string("location","location","location",
-                        NULL, G_PARAM_READWRITE)); // CHECKME
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BYTESPERREAD,
-    g_param_spec_int("bytesperread","bytesperread","bytesperread",
-                     G_MININT,G_MAXINT,0,G_PARAM_READWRITE)); // CHECKME
+  gst_element_install_std_props (
+	  GST_ELEMENT_CLASS (klass),
+	  "location",     ARG_LOCATION,     G_PARAM_READWRITE,
+	  "bytesperread", ARG_BYTESPERREAD, G_PARAM_READWRITE,
+	  NULL);
 
   gobject_class->set_property = gst_httpsrc_set_property;
   gobject_class->get_property = gst_httpsrc_get_property;

test/Makefile.am

 # FIXME FIXME
 
+INCLUDES = $(LIBGST_CFLAGS) $(XML_CFLAGS) $(GLIB_CFLAGS) -I..
+
 if HAVE_GNOME
 GNOME_PROGS = spectrum wave mp1parse videotest aviparse \
 		videotest2 video2mp1 dvshow dv2mp1 \