update singledecodebin to latest version from pitivi

README

@@ -1,4 +1,9 @@
-oxgt-tools
+oxtools
+
+    depends on gstreamer 0.10.30 or newer
+    
+    on ubuntu 10.04 you need
+    sudo add-apt-repository ppa:gstreamer-developers/ppa
 
 Tools
   oxframe

oxgst/imagesink.py

@@ -9,9 +9,6 @@ pygst.require("0.10")
 import gst
 import Image
 
-from singledecodebin import SingleDecodeBin
-
-
 class ImageSink(gst.BaseSink):
 
     __gsignals__ = {

oxgst/singledecodebin.py

@@ -28,6 +28,38 @@ Single-stream queue-less decodebin
 import gobject
 import gst
 
+
+class CachedFactoryList(object):
+    def __init__(self, factoryFilter=None):
+        self._factoryFilter = factoryFilter
+        self._factories = None
+        self._registry = gst.registry_get_default()
+        self._registry.connect("feature-added", self._registryFeatureAddedCb)
+
+    def get(self):
+        if self._factories is None:
+            self._buildFactories()
+
+        return self._factories
+
+    def _buildFactories(self):
+        # build the cache
+        #log.debug("utils", "Getting factories list")
+        factories = self._registry.get_feature_list(gst.ElementFactory)
+        if self._factoryFilter is not None:
+            #log.debug("utils", "filtering")
+            factories = filter(self._factoryFilter, factories)
+
+        #log.debug("utils", "Sorting by rank")
+        factories.sort(key=lambda factory: factory.get_rank(), reverse=True)
+        self._factories = factories
+        #log.debug("utils", "Cached factories is now %r", self._factories)
+
+    def _registryFeatureAddedCb(self, registry, feature):
+        # invalidate the cache
+        #log.warning("utils", "New feature added, invalidating cached factories")
+        self._factories = None
+
 def is_raw(caps):
     """ returns True if the caps are RAW """
     rep = caps.to_string()
@@ -37,7 +69,28 @@ def is_raw(caps):
             return True
     return False
 
+def factoryFilter(factory):
+    if factory.get_rank() < 64 :
+        return False
+
+    klass = factory.get_klass()
+    for cat in ("Demuxer", "Decoder", "Parse"):
+        if cat in klass:
+            return True
+
+    return False
+
+_factoryCache = CachedFactoryList(factoryFilter)
+
 class SingleDecodeBin(gst.Bin):
+    """
+    A variant of decodebin.
+
+    * Only outputs one stream
+    * Doesn't contain any internal queue
+    """
+
+    QUEUE_SIZE = 1 * gst.SECOND
 
     __gsttemplates__ = (
         gst.PadTemplate ("sinkpadtemplate",
@@ -49,11 +102,13 @@ class SingleDecodeBin(gst.Bin):
                          gst.PAD_SOMETIMES,
                          gst.caps_new_any())
         )
-    def __init__(self, caps=None, uri=None, *args, **kwargs):
+
+    def __init__(self, caps=None, uri=None, stream=None, *args, **kwargs):
         gst.Bin.__init__(self, *args, **kwargs)
         if not caps:
             caps = gst.caps_new_any()
         self.caps = caps
+        self.stream = stream
         self.typefind = gst.element_factory_make("typefind", "internal-typefind")
         self.add(self.typefind)
 
@@ -63,7 +118,11 @@ class SingleDecodeBin(gst.Bin):
             self.log("created urisrc %s / %r" % (self.urisrc.get_name(),
                                                  self.urisrc))
             self.add(self.urisrc)
-            self.urisrc.link(self.typefind)
+            # Set the blocksize to 512kbytes, this will only matter for push-based sources
+            if hasattr(self.urisrc.props, "blocksize"):
+                self.urisrc.props.blocksize = 524288
+            self.urisrc.link_pads_full("src", self.typefind, "sink",
+                                       gst.PAD_LINK_CHECK_NOTHING)
         else:
             self._sinkpad = gst.GhostPad("sink", self.typefind.get_pad("sink"))
             self._sinkpad.set_active(True)
@@ -77,8 +136,10 @@ class SingleDecodeBin(gst.Bin):
 
         self._validelements = [] #added elements
 
-        self._factories = self._getSortedFactoryList()
+        self.debug("stream:%r" % self.stream)
 
+        self.pending_newsegment = False
+        self.eventProbeId = None
 
     ## internal methods
 
@@ -88,23 +149,6 @@ class SingleDecodeBin(gst.Bin):
         element.connect("pad-added", self._dynamicPadAddedCb)
         element.connect("no-more-pads", self._dynamicNoMorePadsCb)
 
-    def _getSortedFactoryList(self):
-        """
-        Returns the list of demuxers, decoders and parsers available, sorted
-        by rank
-        """
-        def myfilter(fact):
-            if fact.get_rank() < 64 :
-                return False
-            klass = fact.get_klass()
-            if not ("Demuxer" in klass or "Decoder" in klass or "Parse" in klass):
-                return False
-            return True
-        reg = gst.registry_get_default()
-        res = [x for x in reg.get_feature_list(gst.ElementFactory) if myfilter(x)]
-        res.sort(lambda a, b: int(b.get_rank() - a.get_rank()))
-        return res
-
     def _findCompatibleFactory(self, caps):
         """
         Returns a list of factories (sorted by rank) which can take caps as
@@ -112,7 +156,7 @@ class SingleDecodeBin(gst.Bin):
         """
         self.debug("caps:%s" % caps.to_string())
         res = []
-        for factory in self._factories:
+        for factory in _factoryCache.get():
             for template in factory.get_static_pad_templates():
                 if template.direction == gst.PAD_SINK:
                     intersect = caps.intersect(template.static_caps.get())
@@ -153,6 +197,35 @@ class SingleDecodeBin(gst.Bin):
         for pad in to_connect:
             self._closePadLink(element, pad, pad.get_caps())
 
+    def _isDemuxer(self, element):
+        if not 'Demux' in element.get_factory().get_klass():
+            return False
+
+        potential_src_pads = 0
+        for template in element.get_pad_template_list():
+            if template.direction != gst.PAD_SRC:
+                continue
+
+            if template.presence == gst.PAD_REQUEST or \
+                    "%" in template.name_template:
+                potential_src_pads += 2
+                break
+            else:
+                potential_src_pads += 1
+
+        return potential_src_pads > 1
+
+    def _plugDecodingQueue(self, pad):
+        queue = gst.element_factory_make("queue")
+        queue.props.max_size_time = self.QUEUE_SIZE
+        queue.props.max_size_buffers = 3
+        self.add(queue)
+        queue.sync_state_with_parent()
+        pad.link_full(queue.get_pad("sink"), gst.PAD_LINK_CHECK_NOTHING)
+        pad = queue.get_pad("src")
+
+        return pad
+
     def _tryToLink1(self, source, pad, factories):
         """
         Tries to link one of the factories' element to the given pad.
@@ -162,6 +235,10 @@ class SingleDecodeBin(gst.Bin):
         self.debug("source:%s, pad:%s , factories:%r" % (source.get_name(),
                                                          pad.get_name(),
                                                          factories))
+
+        if self._isDemuxer(source):
+            pad = self._plugDecodingQueue(pad)
+
         result = None
         for factory in factories:
             element = factory.create()
@@ -174,6 +251,7 @@ class SingleDecodeBin(gst.Bin):
                 continue
 
             self.add(element)
+            element.set_state(gst.STATE_READY)
             try:
                 pad.link(sinkpad)
             except:
@@ -204,10 +282,17 @@ class SingleDecodeBin(gst.Bin):
         if caps.is_any():
             self.log("type is not know yet, waiting")
             return
-        if caps.intersect(self.caps):
+
+        self.debug("stream %r" % (self.stream))
+        if caps.intersect(self.caps) and (self.stream is None or
+                (self.stream.pad_name == get_pad_id(pad))):
             # This is the desired caps
             if not self._srcpad:
                 self._wrapUp(element, pad)
+        elif is_raw(caps) and pad_compatible_stream(pad, self.stream):
+            self.log ("not the target stream, but compatible")
+            if not self._srcpad:
+                self._wrapUp(element, pad)
         elif is_raw(caps):
             self.log("We hit a raw caps which isn't the wanted one")
             # FIXME : recursively remove everything until demux/typefind
@@ -232,13 +317,64 @@ class SingleDecodeBin(gst.Bin):
         if self._srcpad:
             return
         self._markValidElements(element)
-        self._removeUnusedElements(self.typefind)
+        gobject.idle_add(self._removeUnusedElements, self.typefind)
-        self.log("ghosting pad %s" % pad.get_name)
+        if pad.props.caps is not None:
+            caps = pad.props.caps
+        else:
+            caps = pad.get_caps()
+
         self._srcpad = gst.GhostPad("src", pad)
         self._srcpad.set_active(True)
+
+        if caps.is_fixed():
+            self._exposePad(target=pad)
+        else:
+            self._blockPad(target=pad)
+
+    def _exposePad(self, target):
+        self.log("ghosting pad %s" % target.get_name())
         self.add_pad(self._srcpad)
         self.post_message(gst.message_new_state_dirty(self))
 
+    def _blockPad(self, target):
+        # don't pass target as an argument to set_blocked_async. Avoids
+        # triggering a bug in gst-python where pad_block_destroy_data calls
+        # CPython without acquiring the GIL
+        self._target = target
+        self._eventProbeId = target.add_event_probe(self._padEventCb)
+        self._srcpad.set_blocked_async(True, self._padBlockedCb)
+
+    def _unblockPad(self, target):
+        target.remove_event_probe(self._eventProbeId)
+        self._eventProbeId = None
+        self._srcpad.set_blocked_async(False, self._padBlockedCb)
+
+    def _padBlockedCb(self, ghost, blocked):
+        if not blocked:
+            if self.pending_newsegment is not None:
+                self._srcpad.push_event(self.pending_newsegment)
+                self.pending_newsegment = None
+            return
+
+        self._exposePad(target=self._target)
+        self._unblockPad(target=self._target)
+
+    def _padEventCb(self, pad, event):
+        if event.type == gst.EVENT_TAG:
+            self.debug("dropping TAG event")
+            return False
+
+        if event.type != gst.EVENT_NEWSEGMENT:
+            self.warning("first event: %s is not a NEWSEGMENT, bailing out" %
+                    event)
+            self._exposePad(target=pad)
+            self._unblockPad(target=pad)
+            return True
+
+        self.debug("stored pending newsegment")
+        self.pending_newsegment = event
+        return False
+
     def _markValidElements(self, element):
         """
         Mark this element and upstreams as valid
@@ -256,10 +392,11 @@ class SingleDecodeBin(gst.Bin):
         """
         Remove unused elements connected to srcpad(s) of element
         """
-        self.log("element:%s" % element)
+        self.log("element:%r" % element)
-        for pad in element.src_pads():
+        for pad in list(element.src_pads()):
             if pad.is_linked():
                 peer = pad.get_peer().get_parent()
+                if isinstance(peer, gst.Element):
                     self._removeUnusedElements(peer)
                     if not peer in self._validelements:
                         self.log("removing %s" % peer.get_name())
@@ -272,6 +409,7 @@ class SingleDecodeBin(gst.Bin):
         if self._srcpad:
             self.remove_pad(self._srcpad)
         self._srcpad = None
+        self._target = None
         for element in self._validelements:
             element.set_state(gst.STATE_NULL)
             self.remove(element)
@@ -282,7 +420,7 @@ class SingleDecodeBin(gst.Bin):
     def do_change_state(self, transition):
         self.debug("transition:%r" % transition)
         res = gst.Bin.do_change_state(self, transition)
-        if transition in [gst.STATE_CHANGE_PAUSED_TO_READY, gst.STATE_CHANGE_READY_TO_NULL]:
+        if transition == gst.STATE_CHANGE_PAUSED_TO_READY:
             self._cleanUp()
         return res