After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 732863 - Adding the properties of internal uridecodebin's source to GESVideoUriSource
Adding the properties of internal uridecodebin's source to GESVideoUriSource
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gst-editing-services
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-07-07 18:00 UTC by César Fabián Orccón Chipana
Modified: 2014-12-13 01:50 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Add GESVideoUriSource's internal source properties to it. (1.42 KB, patch)
2014-07-07 18:02 UTC, César Fabián Orccón Chipana 		rejected Details | Review

Description César Fabián Orccón Chipana 2014-07-07 18:00:30 UTC 
Hello. I have a patch which allows the user of GES set/get the properties of the source the GESVideoUriSource's uridecodebin creates. These properties are *only* available when the uridecodebin has setup its source.

You could do something like
source.get_child_property("property-of-the-internal-source")
source.set_child_property("property-of-the-internal-source")

PD: I'm not sure if this is the correct way to handle "const gchar **' variables since the function "ges_track_element_add_children_props" receives "char gchar **" and not "const gchar **". However, it works.
Comment 1 César Fabián Orccón Chipana 2014-07-07 18:02:15 UTC 
Created attachment 280080 [details] [review]
Add GESVideoUriSource's internal source properties to it.
Comment 2 Thibault Saunier 2014-07-07 18:41:24 UTC 
Review of attachment 280080 [details] [review]:

This is not really a solution as we would need those properties to be exposed all the time for it to be useful for users.

Moreover, we really want to expose *only* the properties that are useful to us.

I think you want to actually use the GESMultiFileSrc class (renaming it to GESImageSequence) and expose only the properties that are meaningfull in there.