GNOME Bugzilla – Bug 708889
Add "smart-properties" property in playbin
Last modified: 2013-09-27 08:24:10 UTC
Overview: I just wanted to set some properties to demuxer or parser or other elements in uridecodebin before PAUSE state. For this, I had to find a element that I wanted to set one by one and I could set a value of property using bus message. So, I Added "smart-properties" property in playbin. This property makes to be possible that all of elements in uridecodebin can set directly their's properties while auto-plugging. A property should be comprised of key and value and split by ':' delimiter. All of properties should be split by ',' delimiter. Expected Results: 1. gst-launch1.0 playbin uri="xxx" smart-properties="program-number:3,emit-stats:TRUE" 2. It is stored to hash table after parsing. 3. While auto-plugging, if there is a matched name of property in a element that is from uridecodebin with hash table, the element starts to set a value of property.
Created attachment 255896 [details] [review] this is a patch file.
Thanks for the patch. I think this is a duplicate of bug #578933 for all practical purposes, even if the proposed solution is a bit different. I don't think a property like this is a good idea, it has lots of problems, such as not being able to restrict which elements to apply it to, properties with the same name but different types on different elements, things like that. Also, the GstChildProxy interface would probably be a better fit for this in general. Lastly, even if a property-based approach was suitable here, the name is not ideal IMHO ("smart" doesn't really mean anything, it doesn't tell anyone anything in addition to -properties) (but that's not so important, just mentioning it while we're at it :)) *** This bug has been marked as a duplicate of bug 578933 ***