gnomemm r1505 - in gstreamermm/trunk: . gstreamer/src

[jaalburqu svn gnome org]

Author: jaalburqu
Date: Mon May 19 02:02:57 2008
New Revision: 1505
URL: http://svn.gnome.org/viewvc/gnomemm?rev=1505&view=rev

Log:
2008-05-18  Josà Alburquerque  <jaalburqu svn gnome org>

	* gstreamer/src/caps.hg: Removed references to Gst::Structure
	ownership

	* gstreamer/src/element.hg:
	* gstreamer/src/gst_enums.defs: Added TODO about making it possible
	for StateChange to be wrapped

	* gstreamer/src/elementfactory.hg: 
	* gstreamer/src/gst_docs_override.xml: Wrapped create_named_element();
	Overrode gst_element_factory_make docs so that mention of NULL name
	parameter is not included

Modified:
   gstreamermm/trunk/ChangeLog
   gstreamermm/trunk/gstreamer/src/caps.hg
   gstreamermm/trunk/gstreamer/src/element.hg
   gstreamermm/trunk/gstreamer/src/elementfactory.hg
   gstreamermm/trunk/gstreamer/src/gst_docs_override.xml
   gstreamermm/trunk/gstreamer/src/gst_enums.defs

Modified: gstreamermm/trunk/gstreamer/src/caps.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/caps.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/caps.hg	Mon May 19 02:02:57 2008
@@ -85,16 +85,13 @@
   _WRAP_METHOD(void append(const Glib::RefPtr<Caps>& caps), gst_caps_append)
   _WRAP_METHOD(void merge(const Glib::RefPtr<Caps>& caps), gst_caps_merge)
 
-  //TODO: This talk of ownership is strange. Say what it really means somewhere. murrayc.
-  /** Appends a structure to caps. The structure is not copied. This Caps becomes the
-   * owner of @a structure.
+  /** Appends a structure to caps.
    *
    * @param structure the Gst::Structure to append
    */
   void append_structure(const Structure& structure);
 
-  /** Appends a structure to caps if its not already expressed by caps. The
-   * structure is not copied. This Caps becomes the owner of @a structure.
+  /** Appends a structure to caps if its not already expressed by caps.
    *
    * @param structure the Gst::Structure to merge
    */

Modified: gstreamermm/trunk/gstreamer/src/element.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/element.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/element.hg	Mon May 19 02:02:57 2008
@@ -48,7 +48,8 @@
 
 _WRAP_ENUM(ElementFlags, GstElementFlags)
 
-//TODO: Why not use _WRAP_ELEM() here?
+//TODO: Fix gmmproc so that definition in gst_enums.defs does not cause errors
+//and this can be wrapped with _WRAP_ENUM.
 enum StateChange
 {
   STATE_CHANGE_NULL_TO_READY = GST_STATE_CHANGE_NULL_TO_READY,

Modified: gstreamermm/trunk/gstreamer/src/elementfactory.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/elementfactory.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/elementfactory.hg	Mon May 19 02:02:57 2008
@@ -53,9 +53,8 @@
   _WRAP_METHOD(static Glib::RefPtr<ElementFactory> find(const Glib::ustring& name), gst_element_factory_find)
 
   //Note that name can't be NULL here, though it seems like gstreamer should allow that as for gst_element_factory_make().
-  //TODO: Wrap, but with a different name to avoid the clash: _WRAP_METHOD(Glib::RefPtr<Element> create_element(const Glib::ustring& name), gst_element_factory_create)
+  _WRAP_METHOD(Glib::RefPtr<Element> create_named_element(const Glib::ustring& name), gst_element_factory_create)
 
-  //TODO: Customize documentatoin to avoid mention of NULL for name.
   _WRAP_METHOD(static Glib::RefPtr<Element> create_element(const Glib::ustring& factory_name, const Glib::ustring& name), gst_element_factory_make)
   
   /** Create a new element of the type defined by the given element factory, 

Modified: gstreamermm/trunk/gstreamer/src/gst_docs_override.xml
==============================================================================
--- gstreamermm/trunk/gstreamer/src/gst_docs_override.xml	(original)
+++ gstreamermm/trunk/gstreamer/src/gst_docs_override.xml	Mon May 19 02:02:57 2008
@@ -18,4 +18,24 @@
 </return>
 </function>
 
+<function name="gst_element_factory_make">
+<description>
+Create a new element of the type defined by the given element factory.
+
+
+</description>
+<parameters>
+<parameter name="factoryname">
+<parameter_description> a named factory to instantiate
+</parameter_description>
+</parameter>
+<parameter name="name">
+<parameter_description> name of new element
+</parameter_description>
+</parameter>
+</parameters>
+<return> new #GstElement or NULL if unable to create element
+</return>
+</function>
+
 </root>

Modified: gstreamermm/trunk/gstreamer/src/gst_enums.defs
==============================================================================
--- gstreamermm/trunk/gstreamer/src/gst_enums.defs	(original)
+++ gstreamermm/trunk/gstreamer/src/gst_enums.defs	Mon May 19 02:02:57 2008
@@ -9,6 +9,7 @@
 ;; MessageType
 ;; PadFlags
 ;; PadTemplateFlags
+;; StateChange
 
 ;; From /opt/usr-jhbuild/include/gstreamer-0.10/gst/gstbin.h
 
@@ -153,10 +154,18 @@
   )
 )
 
+;; TODO: See why gmmproc reports errors when values of following enum are
+;; commented in.
 (define-enum-extended StateChange
   (in-module "Gst")
   (c-name "GstStateChange")
   (values
+;;    '("null-to-ready" "GST_STATE_CHANGE_NULL_TO_READY" "(GST_STATE_NULL<<3) | GST_STATE_READY")
+;;    '("ready-to-paused" "GST_STATE_CHANGE_READY_TO_PAUSED" "(GST_STATE_READY<<3) | GST_STATE_PAUSED")
+;;    '("paused-to-playing" "GST_STATE_CHANGE_PAUSED_TO_PLAYING" "(GST_STATE_PAUSED<<3) | GST_STATE_PLAYING")
+;;    '("playing-to-paused" "GST_STATE_CHANGE_PLAYING_TO_PAUSED" "(GST_STATE_PLAYING<<3) | GST_STATE_PAUSED")
+;;    '("paused-to-ready" "GST_STATE_CHANGE_PAUSED_TO_READY" "(GST_STATE_PAUSED<<3) | GST_STATE_READY")
+;;    '("ready-to-null" "GST_STATE_CHANGE_READY_TO_NULL" "(GST_STATE_READY<<3) | GST_STATE_NULL")
   )
 )