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

[jaalburqu svn gnome org]

Author: jaalburqu
Date: Thu Mar  6 02:31:08 2008
New Revision: 1386
URL: http://svn.gnome.org/viewvc/gnomemm?rev=1386&view=rev

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

	* gstreamer/src/caps.ccg:
	* gstreamer/src/caps.hg: Added gst_caps_new_empty() as new function in
	_CLASS_OPAQUE_REFCOUNTED which automatically generates create()
	method.  Commented out create_full() until we can figure out how to
	have a list of Structures for parameter
	* gstreamer/src/clock.hg: Removed unused _CTOR_DEFAULT
	* gstreamer/src/element.ccg:
	* gstreamer/src/element.hg: Made get_compatible_{pad,pad_template}()
	methods not throw exceptions because the returned RefPtr<> will be
	NULL if none is found.  Used const Glib::ustring& in
	link_pads_filtered() method
	* gstreamer/src/plugin.hg: Removed completed TODO
	* gstreamer/src/tagsetter.hg: Typos

Modified:
   gstreamermm/trunk/ChangeLog
   gstreamermm/trunk/gstreamer/src/caps.ccg
   gstreamermm/trunk/gstreamer/src/caps.hg
   gstreamermm/trunk/gstreamer/src/clock.hg
   gstreamermm/trunk/gstreamer/src/element.ccg
   gstreamermm/trunk/gstreamer/src/element.hg
   gstreamermm/trunk/gstreamer/src/plugin.hg
   gstreamermm/trunk/gstreamer/src/tagsetter.hg

Modified: gstreamermm/trunk/gstreamer/src/caps.ccg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/caps.ccg	(original)
+++ gstreamermm/trunk/gstreamer/src/caps.ccg	Thu Mar  6 02:31:08 2008
@@ -26,12 +26,6 @@
 {
 
 Glib::RefPtr<Caps>
-Caps::create()
-{
-  return Glib::wrap(gst_caps_new_empty());
-}
-
-Glib::RefPtr<Caps>
 Caps::create_any()
 {
   return Glib::wrap(gst_caps_new_any());
@@ -52,11 +46,13 @@
   return result;
 }
 
+/*
 Glib::RefPtr<Caps>
 Caps::create_full(Structure& structure)
 {
   return Glib::wrap(gst_caps_new_full(structure.gobj()));
 }
+*/
 
 /*
   This method is implemented because gst_caps_set_simple is a variable argument

Modified: gstreamermm/trunk/gstreamer/src/caps.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/caps.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/caps.hg	Thu Mar  6 02:31:08 2008
@@ -30,17 +30,15 @@
 
 class Caps 
 {
-  _CLASS_OPAQUE_REFCOUNTED(Caps, GstCaps, NONE, gst_caps_ref, gst_caps_unref)
+  _CLASS_OPAQUE_REFCOUNTED(Caps, GstCaps, gst_caps_new_empty, gst_caps_ref, gst_caps_unref)
   _IGNORE(gst_caps_ref, gst_caps_unref)
 
 public:
-  //TODO: Why doesn't this use _WRAP_CREATE()?
-  static Glib::RefPtr<Caps> create();
   static Glib::RefPtr<Caps> create_any();
   static Glib::RefPtr<Caps> create_simple(const Glib::ustring& media_type);
 
   //TODO: Parameter should be a list of Structures
-  static Glib::RefPtr<Caps> create_full(Structure& structure);
+  //static Glib::RefPtr<Caps> create_full(Structure& structure);
 
   _WRAP_METHOD(Glib::RefPtr<Caps> copy() const, gst_caps_copy)
   _WRAP_METHOD(Glib::RefPtr<Caps> copy_nth(guint nth), gst_caps_copy_nth)

Modified: gstreamermm/trunk/gstreamer/src/clock.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/clock.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/clock.hg	Thu Mar  6 02:31:08 2008
@@ -53,11 +53,6 @@
 {
   _CLASS_GOBJECT(Clock, GstClock, GST_CLOCK, Object, GstObject)
 
-protected:
-  //TODO: What C function does this correspond to?
-  //TODO: Add a create()?
-  _CTOR_DEFAULT()
-
 public:
   _WRAP_METHOD(bool add_observation(ClockTime slave_time, ClockTime master_time, gdouble* result), gst_clock_add_observation)
   _WRAP_METHOD(bool set_master(const Clock& master_clock), gst_clock_set_master)

Modified: gstreamermm/trunk/gstreamer/src/element.ccg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/element.ccg	(original)
+++ gstreamermm/trunk/gstreamer/src/element.ccg	Thu Mar  6 02:31:08 2008
@@ -56,21 +56,13 @@
 Glib::RefPtr<Pad> Element::get_compatible_pad(const Glib::RefPtr<Pad>& pad, const Glib::RefPtr<Caps>& caps)
 {
   GstPad* c_pad = gst_element_get_compatible_pad(gobj(), pad->gobj(), caps->gobj());
-  if(c_pad)
-    return Glib::wrap(c_pad); //gst_element_get_compatible_pad gives us a reference.
-  else
-    throw std::runtime_error("no unlinked Pad in Element '" + get_name() + "' found!");
+  return Glib::wrap(c_pad); //gst_element_get_compatible_pad gives us a reference.
 }
 
 Glib::RefPtr<PadTemplate> Element::get_compatible_pad_template(const Glib::RefPtr<PadTemplate>& padtemplate)
 {
   GstPadTemplate* c_padtemplate = gst_element_get_compatible_pad_template(gobj(), padtemplate->gobj());
-  if(c_padtemplate)
-  {
-    return Glib::wrap(c_padtemplate, true /* take reference */); //gst_element_get_compatible_pad_template() does not give us a reference.
-  }
-  else
-    throw std::runtime_error("no compatible PadTemplate with '" + padtemplate->get_name() + "' in Element '" + get_name() + "' found!");
+  return Glib::wrap(c_padtemplate, true /* take reference */); //gst_element_get_compatible_pad_template() does not give us a reference.
 }
 
 bool Element::query_position(Format& format) const

Modified: gstreamermm/trunk/gstreamer/src/element.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/element.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/element.hg	Thu Mar  6 02:31:08 2008
@@ -92,7 +92,6 @@
   _WRAP_METHOD(Glib::RefPtr<Pad> get_pad(const Glib::ustring& name), gst_element_get_pad)
   _WRAP_METHOD(void create_all_pads(), gst_element_create_all_pads)
 
-  //TODO: Why does this throw an exception? murrayc
   Glib::RefPtr<Pad> get_compatible_pad(const Glib::RefPtr<Pad>& pad, const Glib::RefPtr<Caps>& caps);
   Glib::RefPtr<PadTemplate> get_compatible_pad_template(const Glib::RefPtr<PadTemplate>& padtemplate);
 
@@ -108,8 +107,7 @@
 
   _WRAP_METHOD(void unlink_pads(const Glib::ustring& padname, const Glib::RefPtr<Element>& other_element, const Glib::ustring& other_padname), gst_element_unlink_pads)
 
-  //TODO: Use ustring?:
-  _WRAP_METHOD(bool link_pads_filtered(const char* padname, const Glib::RefPtr<Element>& other_element, const char* others_padname, const Glib::RefPtr<Caps>& filter), gst_element_link_pads_filtered)
+  _WRAP_METHOD(bool link_pads_filtered(const Glib::ustring& padname, const Glib::RefPtr<Element>& other_element, const Glib::ustring& others_padname, const Glib::RefPtr<Caps>& filter), gst_element_link_pads_filtered)
 
   _WRAP_METHOD(bool link_filtered(const Glib::RefPtr<Element>& other_element, const Glib::RefPtr<Caps>& filter), gst_element_link_filtered)
   _WRAP_METHOD(void set_base_time(ClockTime time), gst_element_set_base_time)

Modified: gstreamermm/trunk/gstreamer/src/plugin.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/plugin.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/plugin.hg	Thu Mar  6 02:31:08 2008
@@ -27,8 +27,6 @@
 namespace Gst
 {
 
-//TODO: Deal with struct PluginDesc and SlotInit
-
 //TODO: Edit references below to use gstreamermm classes/methods
 
 /** Plugin â Container for features loaded from a shared object module.

Modified: gstreamermm/trunk/gstreamer/src/tagsetter.hg
==============================================================================
--- gstreamermm/trunk/gstreamer/src/tagsetter.hg	(original)
+++ gstreamermm/trunk/gstreamer/src/tagsetter.hg	Thu Mar  6 02:31:08 2008
@@ -48,7 +48,7 @@
  * tags received from upstream and the tags set by the application via the
  * interface. This can be done like this:
  *
- * //TODO: Add code example from C API docs
+ * TODO: Add code example from C API docs
  */
 class TagSetter : public Glib::Interface
 {
@@ -56,7 +56,7 @@
 
 public:
   //TODO: _WRAP_METHOD(void merge_tags(const Glib::RefPtr<const TagList>& list, TagMergeMode), gst_tag_setter_merge_tags)
-  //TODO: Wrap rest of GstTagStetter method
+  //TODO: Wrap rest of GstTagStetter methods
 };
 
 } // namespace Gst