[Bug 719373] Add multifilesrc support to GES

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=719373
  GStreamer | gst-editing-services | git

Nicolas Dufresne <nicolas.dufresne> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas.dufresne at collabora.
                   |                            |co.uk

--- Comment #2 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2013-11-26 18:43:42 UTC ---
(In reply to comment #1)
> @@ -472,1 +473,4 @@
> -  info = gst_discoverer_discover_uri (discoverer, uri, &lerror);
> +
> +  if (g_str_has_prefix (uri, "multifile://")) {
> +    first_file = g_strdup_printf (&uri[12], 1);
> +    first_file_uri = gst_filename_to_uri (first_file, &lerror);

my nit pick would be tahat instead of using g_strdup_printf, you can simply set

  first_file = uri + 12

This way you save an allocation, still need to free the URI as Mathieu said ;-P

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.