[gstreamer-bugs] [Bug 530417] Add a nicer constructor for GstElements

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=530417

  GStreamer | gst-python | Ver: HEAD CVS

René Stadler changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mail at renestadler.de




------- Comment #5 from René Stadler  2008-06-11 14:28 UTC -------
I propose to revert this and close as NOTABUG.  Johan, you say it's strange to
be forced to use factory functions, but I disagree.  Alessandro pointed out the
obvious case where you are _not_ forced to use a factory to create an element. 
It's clear how it suddenly becomes strange to have the factory name passed to
the constructor here.

I also don't understand how the patch gets rid of the strangeness of the
factory concept anyways.  Instead of gst.element_factory_make you just propose
to call gst.Element instead, but it doesn't become any more pythonic that way. 
It's quite the contrary because gst.Element is a class, and calling a class in
python should return an instance of that class -- not some random subclass
implicitely depending on a constructor argument.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=530417.